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

Re: [PATCH RFC] diff --no-index: teach option to exclude files by pattern

From
Jacob Keller <jacob.e.keller@intel.com>
Date
May 15, 2025, 20:24 UTC
Message-ID
<b7fda1fb-3d4e-4115-bca5-63f2e7829ee6@intel.com>
In-Reply-To
<xmqqtt5lzqxh.fsf@gitster.g>
On 5/15/2025 11:09 AM, Junio C Hamano wrote:
Show 22 quoted lines
> Jacob Keller <jacob.keller@gmail.com> writes:
> 
>> I guess the one weirdness is that pathspecs must come after the first
>> 2 arguments, since we need to find 2 paths first. But this matches the
>> way that treeish must come first in git diff-tree -r takes treeish and
>> then pathspecs, and you can't re-order them arbitrarily either.
>>
>> Does this sound like a reasonable extension to the existing 2 argument
>> form of git diff --no-index?
> 
> Absolutely.
> 
> Or you could even use "--" convention in the examples you would
> write in the documentation, even though you may not absolutely need
> it for the purpose of parsing the command line, to highlight the
> fact that two things to be compared is given and then with an
> optional pathspec after the two things, e.g.,
> 
>  $ git diff --no-index git-1.6.0 git-2.43.0 -- Documentation/
> 
> or something silly like that.
> 

Yea, I'll do that once I get a version with doc. I sent a v2 that works ok, but I think I need some feedback before I fully polish it, since there are a couple of hacks to get things working.

Thanks for the feedback!
Previous: Junio C Hamano
Message 5 of 5 in “diff --no-index: teach option to exclude files by pattern”
  1. diff --no-index: teach option to exclude files by patternJacob Keller, May 14, 2025
  2. Junio C HamanoMay 14, 2025
  3. Jacob KellerMay 15, 2025
  4. Junio C HamanoMay 15, 2025
  5. Jacob KellerMay 15, 2025

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.