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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[ Slow at responding to email, sorry ]

On Mon, 6 Aug 2007, Junio C Hamano wrote:
> 
> That's essentially the patch I sent out in another thread allows
> you to do, and some of these people even Acked them, but there
> is one minor issue.  "git diff" output is paged, and that "not
> up to date" warning, if it is given to stderr, would not be
> usually seen.

I agree. I wouldn't send it to stderr, I'd literally make it part of the 
output.

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..)

		Linus
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux