{"thread":{"id":"55712","subject":"[PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq","startedAt":"2021-05-17T15:14:58Z","lastAt":"2021-05-17T19:46:06Z","messageCount":3,"participants":["Todd Zullinger","Ævar Arnfjörð Bjarmason","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"424747","messageId":"20210517151222.2865093-1-tmz@pobox.com","threadId":"55712","inReplyTo":null,"subject":"[PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq","fromName":"Todd Zullinger","fromEmail":"tmz@pobox.com","sentAt":"2021-05-17T15:12:22Z","receivedAt":"2021-05-17T15:14:58Z","isPatch":true,"sender":{"key":"tmz@pobox.com","avatar":"https://avatars.githubusercontent.com/u/806319?v=4"},"body":"The C_LOCALE_OUTPUT prerequisite was removed in b1e079807b (tests:\nremove last uses of C_LOCALE_OUTPUT, 2021-02-11), where Ævar noted:\n\n    I'm not leaving the prerequisite itself in place for in-flight changes\n    as there currently are none that introduce new tests that rely on it,\n    and because C_LOCALE_OUTPUT is currently a noop on the master branch\n    we likely won't have any new submissions that use it.\n\nOne more use of C_LOCALE_OUTPUT did creep in with 3d1bda6b5b (t7500: add\ntests for --fixup=[amend|reword] options, 2021-03-15).  This causes a\nnumber of the tests to be skipped by default:\n\n    ok 35 # SKIP --fixup=reword: incompatible with --all (missing C_LOCALE_OUTPUT)\n    ok 36 # SKIP --fixup=reword: incompatible with --include (missing C_LOCALE_OUTPUT)\n    ok 37 # SKIP --fixup=reword: incompatible with --only (missing C_LOCALE_OUTPUT)\n    ok 38 # SKIP --fixup=reword: incompatible with --interactive (missing C_LOCALE_OUTPUT)\n    ok 39 # SKIP --fixup=reword: incompatible with --patch (missing C_LOCALE_OUTPUT)\n\nRemove the C_LOCALE_OUTPUT prerequisite from these tests so they are\nnot skipped.\n\nSigned-off-by: Todd Zullinger <tmz@pobox.com>\n---\nI noticed this while testing 2.32.0-rc0.  I grep for skipped tests to help me\ncatch missing requirements in the Fedora packages.\n\n t/t7500-commit-template-squash-signoff.sh | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/t/t7500-commit-template-squash-signoff.sh b/t/t7500-commit-template-squash-signoff.sh\nindex 9092db5fdc..7d02f79c0d 100755\n--- a/t/t7500-commit-template-squash-signoff.sh\n+++ b/t/t7500-commit-template-squash-signoff.sh\n@@ -413,7 +413,7 @@ test_expect_success 'amend! commit allows empty commit msg body with --allow-emp\n '\n \n test_fixup_reword_opt () {\n-\ttest_expect_success C_LOCALE_OUTPUT \"--fixup=reword: incompatible with $1\" \"\n+\ttest_expect_success \"--fixup=reword: incompatible with $1\" \"\n \t\techo 'fatal: reword option of --fixup is mutually exclusive with'\\\n \t\t\t'--patch/--interactive/--all/--include/--only' >expect &&\n \t\ttest_must_fail git commit --fixup=reword:HEAD~ $1 2>actual &&\n"},{"id":"424761","messageId":"87wnrxqphw.fsf@evledraar.gmail.com","threadId":"55712","inReplyTo":"20210517151222.2865093-1-tmz@pobox.com","subject":"Re: [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-05-17T16:42:34Z","receivedAt":"2021-05-17T16:51:16Z","isPatch":true,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Mon, May 17 2021, Todd Zullinger wrote:\n\n> The C_LOCALE_OUTPUT prerequisite was removed in b1e079807b (tests:\n> remove last uses of C_LOCALE_OUTPUT, 2021-02-11), where Ævar noted:\n>\n>     I'm not leaving the prerequisite itself in place for in-flight changes\n>     as there currently are none that introduce new tests that rely on it,\n>     and because C_LOCALE_OUTPUT is currently a noop on the master branch\n>     we likely won't have any new submissions that use it.\n>\n> One more use of C_LOCALE_OUTPUT did creep in with 3d1bda6b5b (t7500: add\n> tests for --fixup=[amend|reword] options, 2021-03-15).  This causes a\n> number of the tests to be skipped by default:\n>\n>     ok 35 # SKIP --fixup=reword: incompatible with --all (missing C_LOCALE_OUTPUT)\n>     ok 36 # SKIP --fixup=reword: incompatible with --include (missing C_LOCALE_OUTPUT)\n>     ok 37 # SKIP --fixup=reword: incompatible with --only (missing C_LOCALE_OUTPUT)\n>     ok 38 # SKIP --fixup=reword: incompatible with --interactive (missing C_LOCALE_OUTPUT)\n>     ok 39 # SKIP --fixup=reword: incompatible with --patch (missing C_LOCALE_OUTPUT)\n>\n> Remove the C_LOCALE_OUTPUT prerequisite from these tests so they are\n> not skipped.\n>\n> Signed-off-by: Todd Zullinger <tmz@pobox.com>\n> ---\n> I noticed this while testing 2.32.0-rc0.  I grep for skipped tests to help me\n> catch missing requirements in the Fedora packages.\n>\n>  t/t7500-commit-template-squash-signoff.sh | 2 +-\n>  1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/t/t7500-commit-template-squash-signoff.sh b/t/t7500-commit-template-squash-signoff.sh\n> index 9092db5fdc..7d02f79c0d 100755\n> --- a/t/t7500-commit-template-squash-signoff.sh\n> +++ b/t/t7500-commit-template-squash-signoff.sh\n> @@ -413,7 +413,7 @@ test_expect_success 'amend! commit allows empty commit msg body with --allow-emp\n>  '\n>  \n>  test_fixup_reword_opt () {\n> -\ttest_expect_success C_LOCALE_OUTPUT \"--fixup=reword: incompatible with $1\" \"\n> +\ttest_expect_success \"--fixup=reword: incompatible with $1\" \"\n>  \t\techo 'fatal: reword option of --fixup is mutually exclusive with'\\\n>  \t\t\t'--patch/--interactive/--all/--include/--only' >expect &&\n>  \t\ttest_must_fail git commit --fixup=reword:HEAD~ $1 2>actual &&\n\nThanks. This is obviously correct.\n\nJunio (added to CC): I also think it makes sense to pull this into rc1,\nsince we have 3d1bda6b5b new in 2.32.0, and will be skipping this part\nof its tests everywhere without this patc.h\n"},{"id":"424778","messageId":"xmqq5yzh6t52.fsf@gitster.g","threadId":"55712","inReplyTo":"87wnrxqphw.fsf@evledraar.gmail.com","subject":"Re: [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-05-17T19:46:01Z","receivedAt":"2021-05-17T19:46:06Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:\n\n> Thanks. This is obviously correct.\n>\n> Junio (added to CC): I also think it makes sense to pull this into rc1,\n> since we have 3d1bda6b5b new in 2.32.0, and will be skipping this part\n> of its tests everywhere without this patc.h\n\nYes, this is an obvious fallout of what we did during this cycle and\nneeds to be fixed up before the release.  Thanks for noticing and\nfixing so quickly.\n\n"}]}