Re: [PATCH v9 34/37] configs: use syntax CONFIG_FOO=n in tools-only_defconfig

[Jerome Forissier]

On 8/23/24 23:30, Tom Rini wrote:
> On Fri, Aug 23, 2024 at 03:48:38PM +0200, Jerome Forissier wrote:
> 
>> The tools-only defconfig causes troubles on MacOSX due to the default
>> C compiler being Clang (LLVM) rather than GCC and more specifically
>> due to [1]. Therefore replace "$ CONFIG_FOO is not set" with the
> 
> "# CONFIG_FOO" not "$ CONFIG_FOO".

Fixed in v10

> 
>> equivalent "CONFIG_FOO=n" using the following command:
>>
>>  $ sed -i -e 's/# \(CONFIG_[^ ]*\) is not set/\1=n/' \
>>        configs/tools-only_defconfig
>>
>> This fixes the tools_only_macOS CI job on GitHub [2].
>>
>> [1] https://github.com/llvm/llvm-project/issues/78778
>> [2] 
>> https://dev.azure.com/u-boot/u-boot/_build/results?buildId=9105&view=results
>>
>> Suggested-by: Tom Rini <tr...@konsulko.com>
>> Signed-off-by: Jerome Forissier <jerome.foriss...@linaro.org>
> 
> Reviewed-by: Tom Rini <tr...@konsulko.com>

Thanks!

-- 
Jerome