Re: [RFC/PATCH 3/2] stash: show combined diff with "stash show"

2014-07-30 Thread Junio C Hamano
On Wed, Jul 30, 2014 at 5:17 PM, Jeff King wrote: > > Like I said, I'm iffy on this part of the series for that reason. But > I'm curious: what do you think should happen in such a use case when > there are staged contents in the index? Right now we completely ignore > them. I think ignoring is a

Re: [RFC/PATCH 3/2] stash: show combined diff with "stash show"

2014-07-30 Thread Jeff King
On Tue, Jul 29, 2014 at 11:13:37AM -0700, Junio C Hamano wrote: > Jeff King writes: > > > ... People might be doing things like "git stash show | git > > apply", and would want to ignore the index content ... > > FWIW, that is exactly how I use "git stash show -p" most of the time. Like I said

Re: [RFC/PATCH 3/2] stash: show combined diff with "stash show"

2014-07-29 Thread Junio C Hamano
Jeff King writes: > ... People might be doing things like "git stash show | git > apply", and would want to ignore the index content ... FWIW, that is exactly how I use "git stash show -p" most of the time. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a messa