Thread (20 messages) flat view 20 messages, 6 authors, 2016-06-15

Re: [PATCH] Several tests: cd inside subshell instead of around

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:49:30

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help