Web lists-archives.com

Re: [PATCH v2 3/3] Add test for describe with a bare repository




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