Phil Hord [off-list ref] writes:
Oof. Sorry. I forgot I have diff.noprefix=true in my local config.
It is a huge timesaver for me when looking at diffs on a console since
I can quickly highlight the filename with a mouse to paste into an
editor.
Sometimes it bites me, though. Usually I notice in the diff, but this
one I was sending with format-patch / send-email.
I guess I'll turn that off in git.git so I don't misfire at you again someday.
I think per-repository configuration might be sufficient for this
particular case (after all, it is project's preference), I wonder if
a more command-specific variant of diff.noprefix so that "log -p"
and "format-patch" can be configured separately would make sense,
something like...
[diff]
noprefix = true
[diff "format-patch"]
noprefix = false
[diff "show"]
noprefix = false
On Thu, Jan 21, 2021 at 6:17 PM Junio C Hamano [off-list ref] wrote:
Phil Hord [off-list ref] writes:
quoted
Oof. Sorry. I forgot I have diff.noprefix=true in my local config.
It is a huge timesaver for me when looking at diffs on a console since
I can quickly highlight the filename with a mouse to paste into an
editor.
Sometimes it bites me, though. Usually I notice in the diff, but this
one I was sending with format-patch / send-email.
I guess I'll turn that off in git.git so I don't misfire at you again someday.
I think per-repository configuration might be sufficient for this
particular case (after all, it is project's preference), I wonder if
a more command-specific variant of diff.noprefix so that "log -p"
and "format-patch" can be configured separately would make sense,
something like...
[diff]
noprefix = true
[diff "format-patch"]
noprefix = false
[diff "show"]
noprefix = false
That seems reasonable. I was trying to think of something clever like
a setting like "auto" that means "noprefix when output is to a tty".
But I still sometimes send a patch to a coworker that I copied from my
console and I then have to remember to add back in the prefixes. So
there is no perfect solution from Git, I think. The correct solution
is to teach my console to skip the prefixes when I double-click the
filename; or to add symlinks at `a` and `b` in my project; or
something else. But these are all more painful than noprefix = true,
so far.
Fwiw - I know this issue has been discussed on the list before; there
are others who feel this itch.
Having config diff.<command>.noprefix seems reasonable as a fix for
format-patch. At first glance it seems like this could get confused
with diff.<driver>.*, but I suppose those settings are all specific to
a driver section, so it would be easy enough to keep them separate.