git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] pretty: allow to override the built-in formats

From
Beat Bolli <dev+git@drbeat.li>
Date
Sep 7, 2020, 05:36 UTC
Message-ID
<8bb68268-8e4c-749e-b2e0-21b38b70c8bf@drbeat.li>
In-Reply-To
<xmqqeene36t7.fsf@gitster.c.googlers.com>
On 06.09.20 23:59, Junio C Hamano wrote:
Show 16 quoted lines
> Denton Liu <liu.denton@gmail.com> writes:
> 
>> Hmm, I'm not sure how I feel about being able to override formats other
>> than "reference".
> 
> Is the idea to introduce a parallel namespace to pretty.<name>?  I
> am not sure why that is a good idea than, say a single variable that
> says "to me, pretty.<name> would override even the built-in names".
> 
> I am not sure how I feel about being able to override built-in
> formats in the first place, though.
> 
> After all, pretty.<name> were introduced so that user-defined ones
> can be invoked with an equal ease as the built-in ones, but
> overriding common understanding among the users of the tool is a
> different story.
I gave a reason for the reference format, at least.

Would you be fine with a patch that just allows to override the reference format (for the stated reasons)?

Previous: Junio C HamanoNext: Junio C Hamano
Message 4 of 10 in “pretty: allow to override the built-in formats”
  1. pretty: allow to override the built-in formatsBeat Bolli, Sep 5, 2020
  2. Denton LiuSep 5, 2020
  3. Junio C HamanoSep 6, 2020
  4. Beat BolliSep 7, 2020
  5. Junio C HamanoSep 7, 2020
  6. Beat BolliSep 7, 2020
  7. Jeff KingSep 8, 2020
  8. Junio C HamanoSep 8, 2020
  9. Jeff KingSep 9, 2020
  10. Junio C HamanoSep 9, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.