git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] t7500: remove non-existant C_LOCALE_OUTPUT prereq

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
May 17, 2021, 16:42 UTC
Message-ID
<87wnrxqphw.fsf@evledraar.gmail.com>
In-Reply-To
<20210517151222.2865093-1-tmz@pobox.com>
On Mon, May 17 2021, Todd Zullinger wrote:
Show 42 quoted lines
> 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

Previous: Todd ZullingerNext: Junio C Hamano
Message 2 of 3 in “t7500: remove non-existant C_LOCALE_OUTPUT prereq”
  1. t7500: remove non-existant C_LOCALE_OUTPUT prereqTodd Zullinger, May 17, 2021
  2. Ævar Arnfjörð BjarmasonMay 17, 2021
  3. Junio C HamanoMay 17, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.