Show 42 quoted lines
> 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.