Patches 5 and 6 look safe, though I haven't tested them. Ramkumar Ramachandra wrote: > --- a/t/t1501-worktree.sh > +++ b/t/t1501-worktree.sh > @@ -48,7 +48,7 @@ test_expect_success 'setup: helper for testing rev-parse' ' > ' > > test_expect_success 'setup: core.worktree = relative path' ' > - unset GIT_WORK_TREE; > + unset GIT_WORK_TREE && On some shells, like /usr/xpg4/bin/sh on Solaris, unset returns nonzero status when the variable passed was already unset. Will this work on such platforms, or does it need to be changed to use sane_unset? -- 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