Re: [PATCH 1/7] remote-testgit: move it into the support directory for t5801
- Date: Mon, 15 Apr 2019 16:08:34 +0900
- From: Junio C Hamano <gitster@xxxxxxxxx>
- Subject: Re: [PATCH 1/7] remote-testgit: move it into the support directory for t5801
"Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx>
> From: Johannes Schindelin <johannes.schindelin@xxxxxx>
> The `git-remote-testgit` script is really only used in
> `t5801-remote-helpers.sh`. It does not even contain any `@@<MAGIC>@@`
> placeholders that would need to be interpolated via `make
> Let's just move it to a new home, decluttering the top-level directory
> and clarifying that this is just a test helper, not an official Git
> command that we would want to ever support.
> @@ -657,8 +656,6 @@ SCRIPT_PERL += git-svn.perl
> SCRIPT_PYTHON += git-p4.py
> -NO_INSTALL += git-remote-testgit
The line lost here was the last one that updated the value of
NO_INSTALL, so we should be able to lose all the mentions of the
make variable now.
> diff --git a/t/t5801-remote-helpers.sh b/t/t5801-remote-helpers.sh
> index aaaa722cca..d04f8007e0 100755
> --- a/t/t5801-remote-helpers.sh
> +++ b/t/t5801-remote-helpers.sh
> @@ -8,6 +8,8 @@ test_description='Test remote-helper import and export commands'
> . ./test-lib.sh
> . "$TEST_DIRECTORY"/lib-gpg.sh
I guess this makes much more sense than having it next to other test
helpers, as t5801 is the home for the remote-helper tests.
Thanks, will queue.