Re: [PATCH] Several tests: cd inside subshell instead of around
- From
Jonathan Nieder <jrnieder@gmail.com>
- Date
- Sep 6, 2010, 19:06 UTC
- Message-ID
- <20100906190655.GG25426@burratino>
- In-Reply-To
- <4C85357A.8090000@web.de>
Jens Lehmann wrote:
> Fixed all places where it was a straightforward change from cd'ing into a > directory and back via "cd .." to a cd inside a subshell.
Thanks, Jens.
> Found these places with "git grep -w "cd \.\.".
I assume that "sed"-ing out all parentheses would make the diff very small and readable.
Do you think it would be a bad idea if I send a follow-on patch that changes code like this:
(cd dir && git update-index --add two && case "`git ls-files`" in two) echo pass two ;; *) echo bad two; exit 1 ;; esac ) &&
to this:
( cd dir && git update-index --add two && case "`git ls-files`" in two) echo pass two ;; *) echo bad two; exit 1 ;; esac )
It would have the benefit of touching all code in the subshell, so we could see the effect on "exit" commands and whatnot.