[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