Subject: Re: [PATCH v2 0/2] meson: prefer '/bin/sh' over PATH lookup

From: Toon Claes <[email-masked]>
Date: Fri, 25 Apr 2025 13:24:19 +0200

Patrick Steinhardt <[email-masked]> writes:

> Hi,
>
> at GitLab, we recently got a couple of bug reports about Git not being
> able to find its shell anymore. The root cause is that with Meson we
> have started to look up the shell via PATH, which may exist on the build
> host, but not on the target host. We have worked around this issue with
> a cross file:
>
>     $ cat >cross.ini <<-EOF
>     [binaries]
>     sh = '/bin/sh'
>     EOF
>     $ meson setup build --cross-file=./cross.ini
>
> But this made me remember the report from Peter [1] that Debian also
> faced this issue. So I decided to address the issue in Meson directly by
> preferring `/bin/sh` over a PATH-based lookup.
>
> Changes in v2:
>   - Simplify how we generate the summary.
>   - Add a comment to explain ordering of the program path.
>   - Link to v1: https://lore.kernel.org/r/[email-masked]
>
> Thanks!
>
> Patrick

Reviewed and looks good to me. It behaves as explained, although I don't
know how to easily test the original error we've been seeing.

-- 
Toon