Re: [PATCH 8/8] ci: use Meson's new `--slice` option

From: Jeff King <peff@peff.net>
Date: Jul 8, 2025, 01:12 UTC
Message-ID: <20250708011249.GA547084@coredump.intra.peff.net>
In-Reply-To: <xmqqldozzge4.fsf@gitster.g>

On Mon, Jul 07, 2025 at 05:16:03PM -0700, Junio C Hamano wrote:

Show 20 quoted lines

> https://github.com/git/git/actions/runs/16129796573/job/45515034049#step:6:17
> 
> unfortunately ends like this:
> 
> Run meson test -C build --no-rebuild --print-errorlogs --slice 0/10
>   meson test -C build --no-rebuild --print-errorlogs --slice 0/10
>   shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"
>   env:
>     DEVELOPER: 1
> usage: meson test [-h] [--maxfail MAXFAIL] [--repeat REPEAT] [--no-rebuild]
>                   [--gdb] [--gdb-path GDB_PATH] [-i] [--list]
>                   [--wrapper WRAPPER] [-C WD] [--suite SUITE] [--no-suite SUITE]
>                   [--no-stdsplit] [--print-errorlogs] [--benchmark]
>                   [--logbase LOGBASE] [-j NUM_PROCESSES] [-v] [-q]
>                   [-t TIMEOUT_MULTIPLIER] [--setup SETUP]
>                   [--test-args TEST_ARGS] [--max-lines MAX_LINES]
>                   [--slice SLICE/NUM_SLICES]
>                   [args ...]
> meson test: error: argument --slice: SLICE is not a positive integer
> Error: Process completed with exit code 1.

I was just digging into this, too. I guess:

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 673b1c44b9..717bd2763b 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -289,7 +289,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        nr: [0, 1, 2, 3, 4, 5, 6, 7, 8, 9]
+        nr: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
     concurrency:
       group: windows-meson-test-${{ matrix.nr }}-${{ github.ref }}
       cancel-in-progress: ${{ needs.ci-config.outputs.skip_concurrent == 'yes' }}

is probably the right fix?

-Peff