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

Re: [PATCH] pathspec: stop --*-pathspecs impact on internal parse_pathspec() uses

From
Jeff King <peff@peff.net>
Date
Oct 26, 2013, 06:39 UTC
Message-ID
<20131026063912.GA24499@sigill.intra.peff.net>
In-Reply-To
<1382753360-32037-1-git-send-email-pclouds@gmail.com>
On Sat, Oct 26, 2013 at 09:09:20AM +0700, Nguyen Thai Ngoc Duy wrote:
Show 6 quoted lines
>  Jeff, how about this?
>  
>  It's similar to your last suggestion (i.e.  relaxing the magic mask
>  about literal magic). In addition, it forces literal magic
>  unconditionally in this case, which I think is the right thing to do.
>  And it will fix other --*-pathspecs as well.

Yeah, I think I follow your reasoning. The problem I saw was just about --literal-pathspec, but the real issue is that "blame" does not want magic pathspecs at all, and your new flag turns that off.

So it fixes both my problem, as well as "git blame -- :(foo)bar".
-Peff
Previous: Nguyễn Thái Ngọc DuyNext: Nguyễn Thái Ngọc Duy
Message 7 of 8 in “[BUG] "git --literal-pathspecs blame" broken in master”
  1. Jeff KingOct 25, 2013
  2. Jeff KingOct 25, 2013
  3. Duy NguyenOct 25, 2013
  4. Jeff KingOct 25, 2013
  5. Duy NguyenOct 25, 2013
  6. pathspec: stop --*-pathspecs impact on internal parse_pathspec() usesNguyễn Thái Ngọc Duy, Oct 26, 2013
  7. Jeff KingOct 26, 2013
  8. pathspec: stop --*-pathspecs impact on internal parse_pathspec() usesNguyễn Thái Ngọc Duy, Oct 26, 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.