[PATCH v9 34/37] configs: use syntax CONFIG_FOO=n in tools-only_defconfig
Jerome Forissier
jerome.forissier at linaro.org
Mon Aug 26 13:48:02 CEST 2024
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 <trini at konsulko.com>
>> Signed-off-by: Jerome Forissier <jerome.forissier at linaro.org>
>
> Reviewed-by: Tom Rini <trini at konsulko.com>
Thanks!
--
Jerome
More information about the U-Boot
mailing list