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

Re: [PATCH] diff-no-index: fix stdin path in subdirectory

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 8, 2025, 16:09 UTC
Message-ID
<xmqqjz3diymi.fsf@gitster.g>
In-Reply-To
<CCF4B0E0-980D-472A-8689-7615FFFE0198@gregoirege.is>
opensource@gregoirege.is writes:
Show 11 quoted lines
> Junio C Hamano <gitster@pobox.com> wrote:
>
>> ... there fundamentally is something wrong here.
>
> Thank you for the thorough response and for explaining the
> reasoning / implementation of the more correct patch.
>
>> Would something more direct like the attached patch work?
>
> I am not familiar with the inner workings of Git at all, but I
> cannot think of any problem with this approach. This works for me!

Ah, what I meant was that I did not test the patch, so I did not know if it solves the problem you observed in your environment and with your development tools, hence I was asking you to apply it and test, like you did when you originally noticed the problem ;-)

Thanks.
Previous: opensource@gregoirege.isNext: opensource@gregoirege.is
Message 4 of 5 in “diff-no-index: fix stdin path in subdirectory”
  1. diff-no-index: fix stdin path in subdirectoryGregoire Geis, Aug 7, 2025
  2. Junio C HamanoAug 7, 2025
  3. opensource@gregoirege.isAug 8, 2025
  4. Junio C HamanoAug 8, 2025
  5. opensource@gregoirege.isAug 9, 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.