{"thread":{"id":"49203","subject":"[PATCH v2 0/1] Make t2024 NO_PERL-safe","startedAt":"2018-08-23T14:41:28Z","lastAt":"2018-08-23T20:59:56Z","messageCount":4,"participants":["Johannes Schindelin via GitGitGadget","Ævar Arnfjörð Bjarmason","Johannes Schindelin"],"isPatch":true,"patchVersion":2,"patchTotal":1},"messages":[{"id":"356366","messageId":"pull.20.v2.git.gitgitgadget@gmail.com","threadId":"49203","inReplyTo":"pull.20.git.gitgitgadget@gmail.com","subject":"[PATCH v2 0/1] Make t2024 NO_PERL-safe","fromName":"Johannes Schindelin via GitGitGadget","fromEmail":"gitgitgadget@gmail.com","sentAt":"2018-08-23T14:41:23Z","receivedAt":"2018-08-23T14:41:28Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"While trying to run the build & test with NO_PERL, I noticed that t2024 had\na failing test case. This patch works around that failing test case by\nskipping it when we know that the error message looks different than that\ntest case would expect.\n\nChanges since v1 (which did not make it to the list due to \nhttps://github.com/gitgitgadget/gitgitgadget/issues/29):\n\n * reworded the commit message slightly.\n\nJohannes Schindelin (1):\n t2024: mark a `checkout -p` test as requiring Perl\n\n t/t2024-checkout-dwim.sh | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\n\nbase-commit: 8d7b558baebe3abbbad4973ce1e1f87a7da17f47\nPublished-As: https://github.com/gitgitgadget/git/releases/tags/pr-20%2Fdscho%2Fcheckout-default-remote-v2\nFetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-20/dscho/checkout-default-remote-v2\nPull-Request: https://github.com/gitgitgadget/git/pull/20\n\nRange-diff vs v1:\n\n 1: 619d7bcc31 ! 1: 8d46b31f5a t2024: mark a `checkout -p` test as requiring Perl\n @@ -3,9 +3,11 @@\n t2024: mark a `checkout -p` test as requiring Perl\n \n A recently-added test case tries to verify that the output of `checkout\n - -p` contains a certain piece of advice. But if Git was built without\n - Perl and therefore lacks support for `git add -i`, the error output\n - contains the hint that `-p` is not even available instead.\n + -p` contains a certain piece of advice.\n +\n + But if Git was built without Perl and therefore lacks support for `git\n + add -i`, the error output contains the hint that `-p` is not even\n + available instead.\n \n Let's just skip that test case altogether if Git was built with NO_PERL.\n \n\n-- \ngitgitgadget\n"},{"id":"356367","messageId":"8d46b31f5ad9925c8d1867c8db4fcbf86480d431.1535035282.git.gitgitgadget@gmail.com","threadId":"49203","inReplyTo":"pull.20.v2.git.gitgitgadget@gmail.com","subject":"[PATCH v2 1/1] t2024: mark a `checkout -p` test as requiring Perl","fromName":"Johannes Schindelin via GitGitGadget","fromEmail":"gitgitgadget@gmail.com","sentAt":"2018-08-23T14:41:25Z","receivedAt":"2018-08-23T14:41:29Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"From: Johannes Schindelin \n\nA recently-added test case tries to verify that the output of `checkout\n-p` contains a certain piece of advice.\n\nBut if Git was built without Perl and therefore lacks support for `git\nadd -i`, the error output contains the hint that `-p` is not even\navailable instead.\n\nLet's just skip that test case altogether if Git was built with NO_PERL.\n\nSigned-off-by: Johannes Schindelin \n---\n t/t2024-checkout-dwim.sh | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/t/t2024-checkout-dwim.sh b/t/t2024-checkout-dwim.sh\nindex 26dc3f1fc0..29e1e25300 100755\n--- a/t/t2024-checkout-dwim.sh\n+++ b/t/t2024-checkout-dwim.sh\n@@ -76,7 +76,8 @@ test_expect_success 'checkout of branch from multiple remotes fails #1' '\n \ttest_branch master\n '\n \n-test_expect_success 'checkout of branch from multiple remotes fails with advice' '\n+test_expect_success NO_PERL \\\n+\t'checkout of branch from multiple remotes fails with advice' '\n \tgit checkout -B master &&\n \ttest_might_fail git branch -D foo &&\n \ttest_must_fail git checkout foo 2>stderr &&\n-- \ngitgitgadget\n"},{"id":"356369","messageId":"877ekhf7rj.fsf@evledraar.gmail.com","threadId":"49203","inReplyTo":"8d46b31f5ad9925c8d1867c8db4fcbf86480d431.1535035282.git.gitgitgadget@gmail.com","subject":"Re: [PATCH v2 1/1] t2024: mark a `checkout -p` test as requiring Perl","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2018-08-23T15:00:00Z","receivedAt":"2018-08-23T15:00:04Z","isPatch":true,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Aug 23 2018, Johannes Schindelin via GitGitGadget wrote:\n\n> From: Johannes Schindelin \n>\n> A recently-added test case tries to verify that the output of `checkout\n> -p` contains a certain piece of advice.\n>\n> But if Git was built without Perl and therefore lacks support for `git\n> add -i`, the error output contains the hint that `-p` is not even\n> available instead.\n>\n> Let's just skip that test case altogether if Git was built with NO_PERL.\n>\n> Signed-off-by: Johannes Schindelin \n> ---\n> t/t2024-checkout-dwim.sh | 3 ++-\n> 1 file changed, 2 insertions(+), 1 deletion(-)\n>\n> diff --git a/t/t2024-checkout-dwim.sh b/t/t2024-checkout-dwim.sh\n> index 26dc3f1fc0..29e1e25300 100755\n> --- a/t/t2024-checkout-dwim.sh\n> +++ b/t/t2024-checkout-dwim.sh\n> @@ -76,7 +76,8 @@ test_expect_success 'checkout of branch from multiple remotes fails #1' '\n> \ttest_branch master\n> '\n>\n> -test_expect_success 'checkout of branch from multiple remotes fails with advice' '\n> +test_expect_success NO_PERL \\\n> +\t'checkout of branch from multiple remotes fails with advice' '\n> \tgit checkout -B master &&\n> \ttest_might_fail git branch -D foo &&\n> \ttest_must_fail git checkout foo 2>stderr &&\n\nThis issue is already fixed in master as 3338e9950e (\"t2024: mark test\nusing \"checkout -p\" with PERL prerequisite\", 2018-08-18).\n"},{"id":"356408","messageId":"nycvar.QRO.7.76.6.1808232259330.73@tvgsbejvaqbjf.bet","threadId":"49203","inReplyTo":"877ekhf7rj.fsf@evledraar.gmail.com","subject":"Re: [PATCH v2 1/1] t2024: mark a `checkout -p` test as requiring Perl","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-08-23T20:59:48Z","receivedAt":"2018-08-23T20:59:56Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Ævar,\n\nOn Thu, 23 Aug 2018, Ævar Arnfjörð Bjarmason wrote:\n\n> \n> On Thu, Aug 23 2018, Johannes Schindelin via GitGitGadget wrote:\n> \n> > From: Johannes Schindelin \n> >\n> > A recently-added test case tries to verify that the output of `checkout\n> > -p` contains a certain piece of advice.\n> >\n> > But if Git was built without Perl and therefore lacks support for `git\n> > add -i`, the error output contains the hint that `-p` is not even\n> > available instead.\n> >\n> > Let's just skip that test case altogether if Git was built with NO_PERL.\n> >\n> > Signed-off-by: Johannes Schindelin \n> > ---\n> > t/t2024-checkout-dwim.sh | 3 ++-\n> > 1 file changed, 2 insertions(+), 1 deletion(-)\n> >\n> > diff --git a/t/t2024-checkout-dwim.sh b/t/t2024-checkout-dwim.sh\n> > index 26dc3f1fc0..29e1e25300 100755\n> > --- a/t/t2024-checkout-dwim.sh\n> > +++ b/t/t2024-checkout-dwim.sh\n> > @@ -76,7 +76,8 @@ test_expect_success 'checkout of branch from multiple remotes fails #1' '\n> > \ttest_branch master\n> > '\n> >\n> > -test_expect_success 'checkout of branch from multiple remotes fails with advice' '\n> > +test_expect_success NO_PERL \\\n> > +\t'checkout of branch from multiple remotes fails with advice' '\n> > \tgit checkout -B master &&\n> > \ttest_might_fail git branch -D foo &&\n> > \ttest_must_fail git checkout foo 2>stderr &&\n> \n> This issue is already fixed in master as 3338e9950e (\"t2024: mark test\n> using \"checkout -p\" with PERL prerequisite\", 2018-08-18).\n\nExcellent,\nDscho"}]}