[Buildroot] [PATCH 1/5] kconfig-package: add support for config fragments
Arnout Vandecappelle
arnout at mind.be
Mon Mar 23 21:41:39 UTC 2015
On 23/03/15 19:31, Yann E. MORIN wrote:
> Jérémy, All,
>
> On 2015-03-23 09:09 +0100, Jeremy Rosen spake thusly:
>> ----- Mail original -----
>>> On 21/03/15 00:11, Yann E. MORIN wrote:
>>>> Floris, All,
>>>>
>> [...]
>>>> use a shell construct here, like:
>>>>
>>>> $$(if $$($(2)_KCONFIG_FRAGMENT_FILES), \
>>>> echo "Unable to perform $(1)-update-config when fragment
>>>> files are set"; exit 1)
>>>
>>> Or perhaps it's better to simply disable the update targets when
>>> fragments are set:
>>>
>>> # Cannot update (def)config with fragments
>>> ifneq ($$($(2)_KCONFIG_FRAGMENT_FILES),)
>>> $(1)-update-config: ...
>>> ...
>>> endif
>>>
>>> [snip]
>>
>> I'm not sure... we are in a case where the user knows that the target
>> is supposed to exist. an unknown target message would be more
>> confusing than the OP's proposed error message. I would not be
>> suprised if someone reported the missing target as a bug here...
>
> Agred, But I guess that what Arnout suggested would be something like:
>
> $(1)-update-config:
> ifneq ($$($(2)_KCONFIG_FRAGMENT_FILES),)
> ...
> else
> echo ERROR.
> enfi
Actually I did mean what I said, but I also agree with Jeremy that it could be
confusing. So +1 to Yann's suggestion.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list