Re: [PATCH 1/2] t5516-fetch-push: fix 'push with dry-run' test

2018-05-10 Thread Junio C Hamano
SZEDER Gábor writes: > In a while-at-it cleanup replacing a 'cd dir && <...> && cd ..' with a > subshell, commit 28391a80a9 (receive-pack: allow deletion of corrupt > refs, 2007-11-29) also moved the assignment of the $old_commit > variable to that subshell. This variable,

[PATCH 1/2] t5516-fetch-push: fix 'push with dry-run' test

2018-05-10 Thread SZEDER Gábor
In a while-at-it cleanup replacing a 'cd dir && <...> && cd ..' with a subshell, commit 28391a80a9 (receive-pack: allow deletion of corrupt refs, 2007-11-29) also moved the assignment of the $old_commit variable to that subshell. This variable, however, is used outside of that subshell as a