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

Re: [regression] Re: git-cat-file --batch reversion; cannot query filenames with spaces

From
Jeff King <peff@peff.net>
Date
Aug 2, 2013, 17:28 UTC
Message-ID
<20130802172804.GB11329@sigill.intra.peff.net>
In-Reply-To
<7vtxj8oxin.fsf@alter.siamese.dyndns.org>
On Fri, Aug 02, 2013 at 09:41:52AM -0700, Junio C Hamano wrote:
Show 9 quoted lines
> > Of the two, I think the latter is more sensible; the former is
> > unnecessarily placing the burden on the user to match "--split" with
> > their use of "%(rest)". The second is pointless without the first.
> >
> > A patch to implement (2) is below.
> 
> As I'd queue this on top of the revert, I had to wrangle it a bit to
> make it relative, i.e. "this resurrects what the other reverted
> patch did but in a weaker/safer form".

Yeah, sorry. After doing the patch I had the thought that maybe the least invasive thing would be the fix rather than the straight revert (we are counting on my assertion that just reverting out part of the series will be OK; I'm pretty sure that is the case, but it is not risk-free, either).

I didn't see the result of your wrangling in pu, but I will keep an eye out to double-check it (unless you did not finish, in which case I am happy to do the wrangling myself).

-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 7 of 11 in “[regression] Re: git-cat-file --batch reversion; cannot query filenames with spaces”
  1. Jonathan NiederAug 2, 2013
  2. Jeff KingAug 2, 2013
  3. Jeff KingAug 2, 2013
  4. Joey HessAug 2, 2013
  5. Brandon CaseyAug 2, 2013
  6. Junio C HamanoAug 2, 2013
  7. Jeff KingAug 2, 2013
  8. Junio C HamanoAug 2, 2013
  9. Jonathan NiederAug 2, 2013
  10. Jeff KingAug 3, 2013
  11. Junio C HamanoAug 2, 2013

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.