Re: [PATCH] install meson for Documentation job

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 18, 2025, 20:40 UTC
Message-ID: <xmqqiko613a0.fsf@gitster.g>
In-Reply-To: <CAOLa=ZRODjYfDXQ8m+hDosV7RBGDWeehzSvsR-+-HCmbS+tAcQ@mail.gmail.com>

Karthik Nayak <karthik.188@gmail.com> writes:

Show 28 quoted lines

> Weird. From the looks of
> https://github.com/git/git/actions/runs/13867157977/job/38808418127, it
> seems like the '$distro' variable in 'ci/install-dependencies.sh' is not
> being correctly initialized. We can see this in the logs of the CI:
>
>   CI setup via install-dependencies.sh
>     + set -e
>     + export TERM=dumb
>     + export MAKEFLAGS=
>     + test true = true
>     + CI_TYPE=github-actions
>     + CI_BRANCH=refs/heads/seen
>     + CI_COMMIT=d5389e37e21d864f40f4167280db799ffe8983c2
>     + echo Linux
>     + tr A-Z a-z
>     + CI_OS_NAME=linux
>     + test macos != linux
>     + CI_REPO_SLUG=git/git
>     + CI_JOB_ID=13867157977
>     + CC=gcc
>     + DONT_SKIP_TAGS=t
>     + cache_dir=/home/runner/none
>     + GIT_TEST_OPTS=--github-workflow-markup
>     + JOBS=10
>     + echo
>     + tr : -
>     + distro=
>     ...

Nice digging.

Show 6 quoted lines

> So in 'ci/lib.sh' we set `distro=$(echo "$CI_JOB_IMAGE" | tr : -)`. So
> this means the distro is based on '$CI_JOB_IMAGE' env variable.
>
> From '.github/workflows/main.yml' we can see that this is not set for
> the 'Documentation' job. I'm not sure why this the way it is though. I
> didn't look into the history.