On Sun, Oct 09, 2016 at 03:24:17PM +0200, René Scharfe wrote: > Offering a way to enable terminal-detection for all color codes of a > format would be useful, but using the existing "auto," prefix for that > would be a behaviour change that could surprise users.
Yeah. In retrospect, it probably would have been saner to make %C(red) a noop when --color is not in effect (either because of --no-color, or more likely when --color=auto is in effect and stdout is not a terminal). But that ship has long since sailed, I think. If we do a revamp of the pretty-formats to bring them more in line with ref-filter (e.g., something like "%(color:red)") maybe that would be an opportunity to make minor adjustments. Though, hmm, it looks like for-each-ref already knows "%(color:red)", and it's unconditional. <sigh> So perhaps we would need to go through some deprecation period or other transition. > --- > Documentation/pretty-formats.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/pretty-formats.txt > b/Documentation/pretty-formats.txt > index a942d57..89e3bc6 100644 > --- a/Documentation/pretty-formats.txt > +++ b/Documentation/pretty-formats.txt > @@ -166,7 +166,8 @@ endif::git-rev-list[] > - '%Cgreen': switch color to green > - '%Cblue': switch color to blue > - '%Creset': reset color > -- '%C(...)': color specification, as described in color.branch.* config > option; > +- '%C(...)': color specification, as described under Values, color in the > + "CONFIGURATION FILE" section of linkgit:git-config[1]; I like the intent here, though I found "Values, color" hard to parse (it was not immediately clear that you mean "the color paragraph of the Values section", as commas are already being used in that sentence for the parenthetical phrase). I'm not sure how to say that succinctly, as we are four levels deep (git-config -> configuration file -> values -> color). Too bad there is no good link syntax for it. Maybe: ...color specification, as described in linkgit:git-config[1] (see the paragraph on colors in the "Values" section, under "CONFIGURATION FILE") or something. -Peff
