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

Re: [PATCH] Add --show-touched option to show "diff --git" line when contents are unchanged

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:43:28

Hi,

On Tue, 7 Aug 2007, Linus Torvalds wrote:
Everybody who takes patches will accept crud afterwards, since the 
normal thing is to email them around, so there's no real downside to 
adding some status output at the end. It shouldn't screw anything up, 
but people will hopefully notice (sure, if you exit the pager without 
looking at it all you wouldn't notice, but that's _already_ true, so..)
But then, you could output the message _twice_: first possibly in-between 
patches ("WARNING: ...") and then at the end.

However, I have the slight suspicion that people will not even notice.  I 
mean, we had bug reports on merge-recursive, where the reporter failed to 
even acknowledge the fact that merge-recursive said that there were 
conflicts, and even listed them.

So I have the slight suspicion that all this will accomplish is "shut the 
darn thing up", and old-timers will have a harder time, since they no 
longer spot easily when they did a Dumb Thing and left the index out of 
sync.

Slightly negative.

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