Re: [PATCH 4/7] commit: fix patch hunk editing with

2014-03-11 Thread Jun Hao
Jeff King peff at peff.net writes: Ah, you're right. I was thinking that our invocation of launch_editor also respected it. It does, but we never get there at all because use_editor is set to 0. So yeah, it really is just needed for the hook. Your patch, even though it is a bigger change,

Re: [PATCH 4/7] commit: fix patch hunk editing with commit -p -m

2014-03-10 Thread Jun Hao
things better overall. However, the simplest fix for this is basically to move the setting of GIT_EDITOR down to after we prepare the index. Jun Hao (cc'd) has been preparing a series for this based on the Bloomberg git hackday a few weeks ago, but it hasn't been sent to the list yet. Commits