Thread (5 messages) 5 messages, 3 authors, 2025-03-20

Re: [PATCH] install meson for Documentation job

From: Karthik Nayak <hidden>
Date: 2025-03-19 16:22:38

Possibly related (same subject, not in this thread)

Patrick Steinhardt [off-list ref] writes:
On Tue, Mar 18, 2025 at 01:40:07PM -0700, Junio C Hamano wrote:
quoted
Karthik Nayak [off-list ref] writes:
quoted
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.
Thanks all for discovering and debugging this, I somehow missed this
thread and haven't been able to keep up with the mailing list for the
last couple days.

I see 4451a164e5a (install meson for Documentation job, 2025-03-14) has
been merged, but as far as I understand we also have to take Karthik's
patch to fix installing dependencies? In case I'm correct, do you plan
to send this as a proper patch, Karthik?
I had a patch locally and after your mail, I did test it out on GitHub
[1]. Will send it here, this should fix the issue and replace
4451a164e5a.

[1]: https://github.com/git/git/actions/runs/13944424140/job/39028258830?pr=1927
Patrick

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help