GNU bug report logs - #20062
[PATCH] diff: add support for --color

Previous Next

Full log

Message #11 received at 20062 <at> debbugs.gnu.org (full text, mbox):

From: Giuseppe Scrivano <gscrivan <at> redhat.com>
To: Eric Blake <eblake <at> redhat.com>
Cc: 20062 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#20062: [PATCH] diff: add support for --color
Date: Mon, 09 Mar 2015 22:21:55 +0100

Eric Blake <eblake <at> redhat.com> writes:

>> +@item auto
>> +@vindex auto @r{color option}
>> +@cindex terminal, using color iff
>> +- Only use color if standard output is a terminal.
>> +@item always
>> +@vindex always @r{color option}
>> +- Always use color.
>> +@end itemize
>> +Specifying @option{--color} and no @var{when} is equivalent to
>> +@option{--color=always}.
>
> Is this the right default?  Some GNU programs default --color to
> --color=auto; others to --color=always.  I'm 50:50 on which default is
> better, but it's worth thinking about rather than just blindly picking
> one, as we have both GNU styles as precedent.

I thought about it as well, and then I decided to go for the same
default as ls but I still believe that auto would be a better one.

If it is fine for everyone, I can rework the patch and change the
default to auto.

Regards,
Giuseppe

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.