Thread (6 messages) flat view 6 messages, 4 authors, 2016-06-15

Re: Incorrect git-blame result if I use full path to file

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: 2016-06-15 22:43:55

Possibly related (same subject, not in this thread)


On Mon, 3 Dec 2007, Linus Torvalds wrote:
On Mon, 3 Dec 2007, Johannes Schindelin wrote:
quoted
I have the slight suspicion that this could break diff --no-index.
Quite possible.
Side note: another issue (for the particular case that Anatol hit) is that 
this patch obviously only helps for commands that actually use 
"get_pathspec()" (usually through doing all the common argument setup 
stuff). So "git log" and friends work fine.

HOWEVER. "git blame" has its own argument parsing that doesn't use any of 
the common routines, and thus the behaviour that Anatol complained about 
isn't fixed at all by the patch.

I think that should be fixed by just making git blame use the standard 
arguments (which in turn may involve having to teach the *other* commands 
about the "-S <revs-file>" and "-L n,m" forms! I think those are why it 
does its own specialized parsing), but obviously git-blame could also be 
tought to just do "get_pathspec()" too.

		Linus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help