git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
* [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq
@ 2021-05-17 15:12 Todd Zullinger
  2021-05-17 16:42 ` Ævar Arnfjörð Bjarmason
  0 siblings, 1 reply; 3+ messages in thread
From: Todd Zullinger @ 2021-05-17 15:12 UTC (permalink / raw)
  To: git
  Cc: Charvi Mendiratta, Christian Couder, Phillip Wood,
	Ævar Arnfjörð Bjarmason

The C_LOCALE_OUTPUT prerequisite was removed in b1e079807b (tests:
remove last uses of C_LOCALE_OUTPUT, 2021-02-11), where Ævar noted:

    I'm not leaving the prerequisite itself in place for in-flight changes
    as there currently are none that introduce new tests that rely on it,
    and because C_LOCALE_OUTPUT is currently a noop on the master branch
    we likely won't have any new submissions that use it.

One more use of C_LOCALE_OUTPUT did creep in with 3d1bda6b5b (t7500: add
tests for --fixup=[amend|reword] options, 2021-03-15).  This causes a
number of the tests to be skipped by default:

    ok 35 # SKIP --fixup=reword: incompatible with --all (missing C_LOCALE_OUTPUT)
    ok 36 # SKIP --fixup=reword: incompatible with --include (missing C_LOCALE_OUTPUT)
    ok 37 # SKIP --fixup=reword: incompatible with --only (missing C_LOCALE_OUTPUT)
    ok 38 # SKIP --fixup=reword: incompatible with --interactive (missing C_LOCALE_OUTPUT)
    ok 39 # SKIP --fixup=reword: incompatible with --patch (missing C_LOCALE_OUTPUT)

Remove the C_LOCALE_OUTPUT prerequisite from these tests so they are
not skipped.

Signed-off-by: Todd Zullinger <tmz@pobox.com>
---
I noticed this while testing 2.32.0-rc0.  I grep for skipped tests to help me
catch missing requirements in the Fedora packages.

 t/t7500-commit-template-squash-signoff.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/t7500-commit-template-squash-signoff.sh b/t/t7500-commit-template-squash-signoff.sh
index 9092db5fdc..7d02f79c0d 100755
--- a/t/t7500-commit-template-squash-signoff.sh
+++ b/t/t7500-commit-template-squash-signoff.sh
@@ -413,7 +413,7 @@ test_expect_success 'amend! commit allows empty commit msg body with --allow-emp
 '
 
 test_fixup_reword_opt () {
-	test_expect_success C_LOCALE_OUTPUT "--fixup=reword: incompatible with $1" "
+	test_expect_success "--fixup=reword: incompatible with $1" "
 		echo 'fatal: reword option of --fixup is mutually exclusive with'\
 			'--patch/--interactive/--all/--include/--only' >expect &&
 		test_must_fail git commit --fixup=reword:HEAD~ $1 2>actual &&

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq
  2021-05-17 15:12 [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq Todd Zullinger
@ 2021-05-17 16:42 ` Ævar Arnfjörð Bjarmason
  2021-05-17 19:46   ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-05-17 16:42 UTC (permalink / raw)
  To: Todd Zullinger
  Cc: git, Charvi Mendiratta, Christian Couder, Phillip Wood, Junio C Hamano


On Mon, May 17 2021, Todd Zullinger wrote:

> The C_LOCALE_OUTPUT prerequisite was removed in b1e079807b (tests:
> remove last uses of C_LOCALE_OUTPUT, 2021-02-11), where Ævar noted:
>
>     I'm not leaving the prerequisite itself in place for in-flight changes
>     as there currently are none that introduce new tests that rely on it,
>     and because C_LOCALE_OUTPUT is currently a noop on the master branch
>     we likely won't have any new submissions that use it.
>
> One more use of C_LOCALE_OUTPUT did creep in with 3d1bda6b5b (t7500: add
> tests for --fixup=[amend|reword] options, 2021-03-15).  This causes a
> number of the tests to be skipped by default:
>
>     ok 35 # SKIP --fixup=reword: incompatible with --all (missing C_LOCALE_OUTPUT)
>     ok 36 # SKIP --fixup=reword: incompatible with --include (missing C_LOCALE_OUTPUT)
>     ok 37 # SKIP --fixup=reword: incompatible with --only (missing C_LOCALE_OUTPUT)
>     ok 38 # SKIP --fixup=reword: incompatible with --interactive (missing C_LOCALE_OUTPUT)
>     ok 39 # SKIP --fixup=reword: incompatible with --patch (missing C_LOCALE_OUTPUT)
>
> Remove the C_LOCALE_OUTPUT prerequisite from these tests so they are
> not skipped.
>
> Signed-off-by: Todd Zullinger <tmz@pobox.com>
> ---
> I noticed this while testing 2.32.0-rc0.  I grep for skipped tests to help me
> catch missing requirements in the Fedora packages.
>
>  t/t7500-commit-template-squash-signoff.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/t/t7500-commit-template-squash-signoff.sh b/t/t7500-commit-template-squash-signoff.sh
> index 9092db5fdc..7d02f79c0d 100755
> --- a/t/t7500-commit-template-squash-signoff.sh
> +++ b/t/t7500-commit-template-squash-signoff.sh
> @@ -413,7 +413,7 @@ test_expect_success 'amend! commit allows empty commit msg body with --allow-emp
>  '
>  
>  test_fixup_reword_opt () {
> -	test_expect_success C_LOCALE_OUTPUT "--fixup=reword: incompatible with $1" "
> +	test_expect_success "--fixup=reword: incompatible with $1" "
>  		echo 'fatal: reword option of --fixup is mutually exclusive with'\
>  			'--patch/--interactive/--all/--include/--only' >expect &&
>  		test_must_fail git commit --fixup=reword:HEAD~ $1 2>actual &&

Thanks. This is obviously correct.

Junio (added to CC): I also think it makes sense to pull this into rc1,
since we have 3d1bda6b5b new in 2.32.0, and will be skipping this part
of its tests everywhere without this patc.h

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq
  2021-05-17 16:42 ` Ævar Arnfjörð Bjarmason
@ 2021-05-17 19:46   ` Junio C Hamano
  0 siblings, 0 replies; 3+ messages in thread
From: Junio C Hamano @ 2021-05-17 19:46 UTC (permalink / raw)
  To: Ævar Arnfjörð Bjarmason
  Cc: Todd Zullinger, git, Charvi Mendiratta, Christian Couder, Phillip Wood

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:

> Thanks. This is obviously correct.
>
> Junio (added to CC): I also think it makes sense to pull this into rc1,
> since we have 3d1bda6b5b new in 2.32.0, and will be skipping this part
> of its tests everywhere without this patc.h

Yes, this is an obvious fallout of what we did during this cycle and
needs to be fixed up before the release.  Thanks for noticing and
fixing so quickly.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-05-17 19:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-17 15:12 [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq Todd Zullinger
2021-05-17 16:42 ` Ævar Arnfjörð Bjarmason
2021-05-17 19:46   ` Junio C Hamano

Code repositories for project(s) associated with this inbox:

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).