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

Re: [PATCH 5/7] ci: make test slicing consistent across Meson/Make

From
Patrick Steinhardt <ps@pks.im>
Date
Feb 19, 2026, 06:18 UTC
Message-ID
<aZarM9CeGjBqC3S3@pks.im>
In-Reply-To
<83168201-9c26-4ece-51e1-1931d4b38443@gmx.de>
On Wed, Feb 18, 2026 at 10:32:20PM +0100, Johannes Schindelin wrote:
Show 24 quoted lines
> Hi Patrick,
> 
> On Wed, 11 Feb 2026, Patrick Steinhardt wrote:
> 
> > In the preceding commit we have adjusted test slicing to be one-based
> > when using the "ci/run-test-slice.sh" script. But we also have an
> > equivalent script for Meson that is still zero-based, which is of course
> > inconsistent.
> > 
> > Adapt the script to be one-based, as well, and adapt the GitHub workflow
> > accordingly. Note that GitLab doesn't yet use the script, so it does not
> > need to be adapted. This will change in the next commit though.
> 
> This seems to require the following to be squashed in:
> 
> -- snip --
> From 677370c8cd3b73358713a0efac6bb573b347d53f Mon Sep 17 00:00:00 2001
> From: Johannes Schindelin <johannes.schindelin@gmx.de>
> Date: Wed, 18 Feb 2026 22:12:27 +0100
> Subject: [PATCH] fixup??? ci: make test slicing consistent across Meson/Make
> 
> This seems to be needed to let the first "win+VS test" job pass. See
> https://github.com/git-for-windows/git/actions/runs/22153194896/job/64053953314#step:5:70
> ("fatal: slice must be in the range 1 <= slice <= 10, got '0'").

Huh, weird, why didn't I spot this myself? Oh, these tests don't execute by default, right? That's why.

Show 24 quoted lines
> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
>  .github/workflows/main.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
> index 412842eb3e2..42585fff779 100644
> --- a/.github/workflows/main.yml
> +++ b/.github/workflows/main.yml
> @@ -240,7 +240,7 @@ jobs:
>        shell: bash
>        env:
>          NO_SVN_TESTS: 1
> -      run: . /etc/profile && ci/run-test-slice.sh ${{matrix.nr}} 10
> +      run: . /etc/profile && ci/run-test-slice.sh $((${{matrix.nr}}+1)) 10
>      - name: print test failures
>        if: failure() && env.FAILED_TEST_ARTIFACTS != ''
>        shell: bash
> -- snap --
> 
> This here run proves that this change fixes the CI failure:
> https://github.com/git-for-windows/git/actions/runs/22157880378/job/64067214717#step:5:70
> 
> Patrick, would you mind adopting this amendment into your patch?
Yup, looks obviously correct. Will squash in, thanks!
Patrick
Previous: Johannes SchindelinNext: Patrick Steinhardt
Message 8 of 12 in “Some assorted fixes for GitLab CI”
  1. 0/7 Some assorted fixes for GitLab CIPatrick Steinhardt, Feb 11, 2026
  2. 1/7 ci: handle failures of test-slice helperPatrick Steinhardt, Feb 11, 2026
  3. 2/7 ci: don't skip smallest test slice in GitLabPatrick Steinhardt, Feb 11, 2026
  4. 3/7 meson: fix MERGE_TOOL_DIR with "--no-bin-wrappers"Patrick Steinhardt, Feb 11, 2026
  5. 4/7 github: fix Meson tests not executing at allPatrick Steinhardt, Feb 11, 2026
  6. 5/7 ci: make test slicing consistent across Meson/MakePatrick Steinhardt, Feb 11, 2026
  7. Johannes SchindelinFeb 18, 2026
  8. Patrick SteinhardtFeb 19, 2026
  9. 6/7 gitlab-ci: use "run-test-slice-meson.sh"Patrick Steinhardt, Feb 11, 2026
  10. 7/7 gitlab-ci: handle failed tests on MSVC+Meson jobPatrick Steinhardt, Feb 11, 2026
  11. Patrick SteinhardtFeb 11, 2026
  12. Junio C HamanoFeb 11, 2026

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.