Re: %C(auto) not working as expected

From: Duy Nguyen <pclouds@gmail.com>
Date: Oct 25, 2016, 13:02 UTC
Message-ID: <CACsJy8AdwpoTRaaPBtmwipRP84Q=kUAabagYeMkHmT2kE3m+pw@mail.gmail.com>
In-Reply-To: <20161025125856.p2paxt7erl2szptv@sigill.intra.peff.net>

On Tue, Oct 25, 2016 at 7:58 PM, Jeff King <peff@peff.net> wrote:

Show 17 quoted lines

> On Tue, Oct 25, 2016 at 07:52:21PM +0700, Duy Nguyen wrote:
>
>> > Yeah, adding a "%C(enable-auto-color)" or something would be backwards
>> > compatible and less painful than using "%C(auto)" everywhere. I do
>> > wonder if anybody actually _wants_ the "always show color, even if
>> > --no-color" behavior. I'm having trouble thinking of a good use for it.
>> >
>> > IOW, I'm wondering if anyone would disagree that the current behavior is
>> > simply buggy.
>>
>> Silence in two weeks. I vote (*) making %(<color-name>) honor --color
>> and turning the %(auto, no-op, for both log family and for-each-ref.
>> We could keep old behavior behind some environment variable if it's
>> not much work so it keeps working while people come here and tell us
>> about their use cases.
>
> Yeah, sorry.

Just to be clear (after re-reading my mail), the unwritten part after "Silence in two weeks" was "so nobody disagreed with you", not "how is your progress?". It's great that you keep working on this regardless :D

-- 
Duy