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

Re: [PATCH] t: avoid alternation (not POSIX) in grep's BRE

From
Jeff King <peff@peff.net>
Date
May 28, 2020, 16:52 UTC
Message-ID
<20200528165245.GA1223396@coredump.intra.peff.net>
In-Reply-To
<xmqqh7w03vnq.fsf@gitster.c.googlers.com>
On Thu, May 28, 2020 at 08:51:21AM -0700, Junio C Hamano wrote:
Show 6 quoted lines
> > I think it may be a good policy to stick to the
> > simplest machine-readable formats for trailers. Likewise I'd suggest
> > using the full sha1-hex for future-proofing in this context.
> 
> Yes, or just roll it into prose like we often do.  Anybody can spot
> many examples from "git log --no-merges" ;-)

Yeah, I was sort of working under the assumption that people found utility in putting that data in a trailer (in which case I'd really suggest doing both; a prose one with subject meant for readers, and a machine-readable annotation).

But I haven't really found a use for "Fixes" in machine-readable format. I don't _mind_ people doing it if they do have a use (and I'd even consider doing it myself if I were shown that it was useful). In the meantime, I don't know if we want to state a project preference against it.

-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 5 of 10 in “t: avoid alternation (not POSIX) in grep's BRE”
  1. t: avoid alternation (not POSIX) in grep's BRECarlo Marcelo Arenas Belón, May 28, 2020
  2. Junio C HamanoMay 28, 2020
  3. Jeff KingMay 28, 2020
  4. Junio C HamanoMay 28, 2020
  5. Jeff KingMay 28, 2020
  6. Junio C HamanoMay 28, 2020
  7. Jeff KingMay 28, 2020
  8. digging into historical commit referencesJeff King, May 29, 2020
  9. Torsten BögershausenMay 29, 2020
  10. t: avoid alternation (not POSIX) in grep's BRECarlo Marcelo Arenas Belón, May 29, 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.