threads / patch / 18824

patchgittutorial: Add global color.ui

Subject: [PATCH] gittutorial: Add global color.ui

## tl;dr

3 messages between Apr 10, 2009 and Apr 13, 2009. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Paolo Ciarrocchi· Apr 10, 2009, 12:16 UTC · lore

I consider color.ui so important that I think it deserves to be mentioned in the gittutorial document. Patch is against pu.

Signed-off-by: Paolo Ciarrocchi <paolo.ciarrocchi@gmail.com>
---
 Documentation/gittutorial.txt |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

Show changes to Documentation/gittutorial.txt +8 −0

diff --git a/Documentation/gittutorial.txt b/Documentation/gittutorial.txt
index c5d5596..6748fda 100644
--- a/Documentation/gittutorial.txt
+++ b/Documentation/gittutorial.txt
@@ -44,6 +44,14 @@ $ git config --global user.name "Your Name Comes Here"
 $ git config --global user.email you@yourdomain.example.com
 ------------------------------------------------
 
+This will make prettier the output of certain commands such as `git diff`
+and `git branch`:
+
+------------------------------------------------
+$ git config --global color.ui auto
+------------------------------------------------
+
+
 
 Importing a new project
 -----------------------

-- 
1.6.2.2.485.ge37347

Paolo Ciarrocchi· Apr 13, 2009, 07:03 UTC · re: Paolo Ciarrocchi · lore

Re: [PATCH] gittutorial: Add global color.ui

Hello, AFAIK the below path has not been commented or applied.

Should I just wait or does it means that is not considered usefull?

Regards,
        Paolo

On 4/10/09, Paolo Ciarrocchi <paolo.ciarrocchi@gmail.com> wrote:

Show 32 quoted lines

> I consider color.ui so important that I think it deserves to be mentioned in
> the gittutorial document.
> Patch is against pu.
>
> Signed-off-by: Paolo Ciarrocchi <paolo.ciarrocchi@gmail.com>
> ---
>  Documentation/gittutorial.txt |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/gittutorial.txt b/Documentation/gittutorial.txt
> index c5d5596..6748fda 100644
> --- a/Documentation/gittutorial.txt
> +++ b/Documentation/gittutorial.txt
> @@ -44,6 +44,14 @@ $ git config --global user.name "Your Name Comes Here"
>  $ git config --global user.email you@yourdomain.example.com
>  ------------------------------------------------
>
> +This will make prettier the output of certain commands such as `git diff`
> +and `git branch`:
> +
> +------------------------------------------------
> +$ git config --global color.ui auto
> +------------------------------------------------
> +
> +
>
>  Importing a new project
>  -----------------------
> --
> 1.6.2.2.485.ge37347
>
>

Junio C Hamano· Apr 13, 2009, 07:29 UTC · re: Paolo Ciarrocchi · lore

Re: [PATCH] gittutorial: Add global color.ui

Paolo Ciarrocchi <paolo.ciarrocchi@gmail.com> writes:

> Should I just wait or does it means that is not considered usefull?

I didn't find it useful myself. Even though it is not the final word around here, I didn't see anybody else agreeing that having color.ui at the begining would improve the tutorial.

Many other things are probably as useful to have as color.ui there, if not more, and we obviously do not want to list all the things some random person found useful and extend the initial part of the tutorial to 100 pages before showing how to create the very first repository to a new person.

So you have to do much better than just saying that you like color.ui. You need to justify that it should trump all the other potentially useful configurations and added there.