On Sun, Jan 27, 2019 at 3:51 AM Sebastian Staudt <koraktor@xxxxxxxxx> wrote: > > This ensures that nothing breaks the basic functionality of describe for > bare repositories. Please note that --broken and --dirty need a working > tree. > > Signed-off-by: Sebastian Staudt <koraktor@xxxxxxxxx> > --- > t/t6120-describe.sh | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/t/t6120-describe.sh b/t/t6120-describe.sh > index 9a6bd1541f..ddd8cc307d 100755 > --- a/t/t6120-describe.sh > +++ b/t/t6120-describe.sh > @@ -409,4 +409,11 @@ test_expect_success 'describe complains about missing object' ' > test_must_fail git describe $ZERO_OID > ' > > +test_expect_success 'describe works from outside repo using --git-dir' " > + BARE_CLONE=$(mktemp -d) && No, keep everything in $TRASH_DIRECTORY so it will be automatically cleaned. > + git clone --bare '$TRASH_DIRECTORY' \$BARE_CLONE >/Users/koraktor/open-source/others/git/t/out && Ehh.. I'm pretty sure I don't have /Users/koraktor on my system :) This looks like just debug code, I think you can drop ">.." part for all commands. > + echo $PWD >/Users/koraktor/open-source/others/git/t/out && > + git --git-dir \$BARE_CLONE describe 2>&1 >/Users/koraktor/open-source/others/git/t/out > +" > + > test_done > -- > 2.20.1 > -- Duy