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

Re: [PATCH] git-add: allow path limiting with -u

From
Junio C Hamano <junkio@cox.net>
Date
May 14, 2007, 01:29 UTC
Message-ID
<7virawp4zo.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<20070514004224.GB6689@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 10 quoted lines
> On Sun, May 13, 2007 at 12:35:24PM +0200, Jakub Narebski wrote:
>
>> The fact that "add --interactive does not take any parameters" is
>> separate issue (which, accidentally, was adressed in the same patch).
>
> I don't generally use add --interactive, but I imagine that
> path-limiting would also make sense there. I think it would be a bit
> harder to implement (and test!), since there are many calls to commands
> which would need the limits. So I will leave that unless somebody really
> cares about it.

I do not use -i myself that much, but due to its "interactive" nature, I would imagine that limiting path upfront is actually counter-productive. During the interactive refining process you might notice that you wanted to handle some other paths you excluded from the command line, but you won't be able to access them if you limit upfront. In other words, command line limiter would only interfere with what the interactive user would really want to do.

Previous: Jeff King
Message 9 of 9 in “git-add: allow path limiting with -u”
  1. git-add: allow path limiting with -uJeff King, May 12, 2007
  2. Junio C HamanoMay 12, 2007
  3. Jakub NarebskiMay 13, 2007
  4. Jeff KingMay 14, 2007
  5. Jakub NarebskiMay 14, 2007
  6. Jeff KingMay 14, 2007
  7. Junio C HamanoMay 14, 2007
  8. Jeff KingMay 14, 2007
  9. Junio C HamanoMay 14, 2007

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.