On Sun, Apr 3, 2016 at 12:20 AM, Johannes Sixt <j6t@xxxxxxxx> wrote: > Am 02.04.2016 um 19:58 schrieb Mehul Jain: >> +for i in --autostash --no-autostash >> +do >> + test_expect_success "pull $i (without --rebase) is illegal" ' >> + test_must_fail git pull $i . copy 2>err && >> + test_i18ngrep "only valid with --rebase" err >> + ' >> +done > > > Hm. If the implementation of test_expect_success uses the variable, too, its > value is lost when the test snippet runs. Fortunately, it does not. > > You can make this code a bit more robust by using double-quotes around the > test code so that $i is expanded before test_expect_success is evaluated. I think that the current format is preferred over the one you suggest. Here[1] Junio has given a descriptive explanation. [1]: http://thread.gmane.org/gmane.comp.version-control.git/283350/focus=284769 Thanks, Mehul -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html