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

2014-03-19 Thread Junio C Hamano
Torsten Bögershausen writes: >> +int run_hook_with_custom_index(const char *index_file, const char *name, >> ...) >> +{ >> +const char *hook_env[3] = { NULL }; >> +char index[PATH_MAX]; > Sorry being late with the review. > > Recently some effort has been put to replace the > "PATH_MAX

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

2014-03-19 Thread Torsten Bögershausen
On 03/18/2014 11:00 AM, Benoit Pierre wrote: Don't change git environment: move the GIT_EDITOR=":" override to the hook command subprocess, like it's already done for GIT_INDEX_FILE. Signed-off-by: Benoit Pierre --- builtin/checkout.c | 8 builtin/clone.c | 4 ++-- b

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

2014-03-18 Thread Benoit Pierre
Don't change git environment: move the GIT_EDITOR=":" override to the hook command subprocess, like it's already done for GIT_INDEX_FILE. Signed-off-by: Benoit Pierre --- builtin/checkout.c | 8 builtin/clone.c | 4 ++-- builtin/commit.c| 35 ++

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

2014-03-15 Thread Benoit Pierre
Don't change git environment: move the GIT_EDITOR=":" override to the hook command subprocess, like it's already done for GIT_INDEX_FILE. Signed-off-by: Benoit Pierre --- builtin/checkout.c | 8 builtin/clone.c | 4 ++-- builtin/commit.c| 35 ++

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

2014-03-11 Thread Junio C Hamano
Benoit Pierre writes: > diff --git a/builtin/checkout.c b/builtin/checkout.c > index 5df3837..da423b2 100644 > --- a/builtin/checkout.c > +++ b/builtin/checkout.c > @@ -53,10 +53,10 @@ struct checkout_opts { > static int post_checkout_hook(struct commit *old, struct commit *new, >

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

2014-03-11 Thread Jeff King
On Tue, Mar 11, 2014 at 06:56:02PM +0100, Benoit Pierre wrote: > According to the original commit, the change to GIT_EDITOR is only > here for hooks: > > commit 406400ce4f69e79b544dd3539a71b85d99331820 > Author: Paolo Bonzini > Date: Tue Feb 5 11:01:45 2008 +0100 > > git-commit: set GIT_E

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

2014-03-11 Thread Benoit Pierre
On Mon, Mar 10, 2014 at 9:07 PM, Jeff King wrote: > On Mon, Mar 10, 2014 at 07:49:34PM +0100, Benoit Pierre wrote: > >> Don't change git environment: move the GIT_EDITOR=":" override to the >> hook command subprocess, like it's already done for GIT_INDEX_FILE. >> [...] > > This is a lot of change,

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

2014-03-10 Thread Jun Hao
Jeff King writes: > On Mon, Mar 10, 2014 at 07:49:34PM +0100, Benoit Pierre wrote: > >> Don't change git environment: move the GIT_EDITOR=":" override to the >> hook command subprocess, like it's already done for GIT_INDEX_FILE. >> >> Signed-off-by: Benoit Pierre >> --- >> builtin/checkout.c

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

2014-03-10 Thread Jeff King
On Mon, Mar 10, 2014 at 07:49:34PM +0100, Benoit Pierre wrote: > Don't change git environment: move the GIT_EDITOR=":" override to the > hook command subprocess, like it's already done for GIT_INDEX_FILE. > > Signed-off-by: Benoit Pierre > --- > builtin/checkout.c| 8 +++ >

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

2014-03-10 Thread Benoit Pierre
Don't change git environment: move the GIT_EDITOR=":" override to the hook command subprocess, like it's already done for GIT_INDEX_FILE. Signed-off-by: Benoit Pierre --- builtin/checkout.c| 8 +++ builtin/clone.c | 4 ++-- builtin/commit.c