From: Toon Claes <toon@iotcl.com>
Date: Thu, 10 Jul 2025 15:25:49 GMT
Subject: Re: [PATCH v3 3/8] meson: improve summary of auto-detected features
Message-ID: <87h5zkcbk2.fsf@iotcl.com>
In-Reply-To: <20250709-b4-pks-meson-cleanups-v3-3-29ab15b9ab85@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

> The summary of auto-detected features prints a boolean for every option
> to tell the user whether or not the feature has been auto-enabled or
> not. This summary can be improved though, as in some cases this boolean
> is derived from a dependency. So if we pass in the dependency directly,
> then Meson knows to both print a boolean and, if the dependency was
> found, it also prints a version number.
>
> Adapt the code accordingly and enable `bool_yn` so that actual booleans
> are formatted similarly to dependencies. Before this change:
>
>   Auto-detected features
>     benchmarks      : true
>     curl            : true
>     expat           : true
>     gettext         : true
>     gitweb          : true
>     iconv           : true
>     pcre2           : true
>     perl            : true
>     python          : true
>
> And after this change, we now see the version numbers as expected:
>
>   Auto-detected features
>     benchmarks      : YES
>     curl            : YES 8.14.1
>     expat           : YES 2.7.1
>     gettext         : YES
>     gitweb          : YES
>     iconv           : YES
>     pcre2           : YES 10.44
>     perl            : YES
>     python          : YES
>
> Note that this change also enables colorization of the boolean options,
> green for "YES" and red for "NO".

Nice, I'm really liking this change.

I've also reviewed the rest of the patches, and all looks good to me.

-- 
Cheers,
Toon