Re: [PATCH 3/3] t/t7106-reset-unborn-branch.sh: Add PERL prerequisite

2013-08-25 Thread Junio C Hamano
Jonathan Nieder jrnie...@gmail.com writes: Kacper Kornet wrote: Signed-off-by: Kacper Kornet drae...@pld-linux.org Thanks. Reviewed-by: Jonathan Nieder jrnie...@gmail.com Here's a style cleanup on top. -- 8 -- Subject: reset test: modernize style Avoid command substitution and pipes

Re: [PATCH 3/3] t/t7106-reset-unborn-branch.sh: Add PERL prerequisite

2013-08-25 Thread Jonathan Nieder
Junio C Hamano wrote: The change to the one that feeds 'y' to reset -p may be a bit too pedantic, as we are not in the business of testing echo y, though. Yeah, that's true. Here's a patch for squashing in. diff --git i/t/t7106-reset-unborn-branch.sh w/t/t7106-reset-unborn-branch.sh index

Re: [PATCH 3/3] t/t7106-reset-unborn-branch.sh: Add PERL prerequisite

2013-08-24 Thread Jonathan Nieder
Kacper Kornet wrote: Signed-off-by: Kacper Kornet drae...@pld-linux.org Thanks. Reviewed-by: Jonathan Nieder jrnie...@gmail.com Here's a style cleanup on top. -- 8 -- Subject: reset test: modernize style Avoid command substitution and pipes to ensure that the exit status from each git