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

Re: [PATCH] apply: strip ./ prefix from --directory argument

From
Patrick Steinhardt <ps@pks.im>
Date
Feb 18, 2026, 14:14 UTC
Message-ID
<aZXJVtgRYQAnxNnE@pks.im>
In-Reply-To
<xmqqzf579kuc.fsf@gitster.g>
On Tue, Feb 17, 2026 at 12:27:39PM -0800, Junio C Hamano wrote:
Show 9 quoted lines
> Patrick Steinhardt <ps@pks.im> writes:
> 
> > While this change here fixes your observed issues, the next person might
> > run into a totally different one. So more generally, I think what we'd
> > rather want to do is to fully normalize the path. How about this
> > instead:
> 
> Sorry, but I am confused.  Why isn't "don't do it then" a good
> answer for a case like this?

I guess that's fair, but especially with command line completion it might be easy to arrive at such paths. That may or may not be a good argument, I'm not sure myself.

Patrick
Previous: Junio C HamanoNext: Joaquim Rocha
Message 4 of 7 in “apply: strip ./ prefix from --directory argument”
  1. apply: strip ./ prefix from --directory argumentJoaquim Rocha via GitGitGadget, Feb 13, 2026
  2. Patrick SteinhardtFeb 17, 2026
  3. Junio C HamanoFeb 17, 2026
  4. Patrick SteinhardtFeb 18, 2026
  5. Joaquim RochaFeb 18, 2026
  6. apply: normalize path in --directory argumentJoaquim Rocha via GitGitGadget, Feb 18, 2026
  7. Junio C HamanoFeb 20, 2026

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.