[PATCH] make devfsd use the new ENABLE_* stuff

Bernhard Fischer rep.nop at aon.at
Sat Oct 8 18:49:34 UTC 2005


On Sat, Oct 08, 2005 at 07:31:06PM +0200, Tito wrote:
>On Saturday 08 October 2005 18:45, you wrote:
>> On Sat, Oct 08, 2005 at 06:15:18PM +0200, Tito wrote:
>
>snip
>
>> Can you please switch the optional features to CONFIG_FEATURE_* while at
>> it?
>> e.g. CONFIG_DEVFSD_MODLOAD -> CONFIG_FEATURE_DEVFSD_MODLOAD
>> 
>
>Hi, 
>in Config.in?
>
>config CONFIG_DEVFSD
>config CONFIG_DEVFSD_MODLOAD
>config CONFIG_DEVFSD_FG_NP
>config CONFIG_DEVFSD_VERBOSE
>
>Like this  ?

Yes, exactly.
>
>config CONFIG_DEVFSD  /* Seems to me this one must stay like it is ??? */
>config CONFIG_FEATURE_DEVFSD_MODLOAD
>config CONFIG_FEATURE_DEVFSD_FG_NP
>config CONFIG_FEATURE_DEVFSD_VERBOSE
>
>and then obviously as you say:
>
>>+/* Busybox stuff */
>>+#if defined(CONFIG_DEVFSD_VERBOSE) || defined(CONFIG_DEBUG)
>
>All these should probably read along the lines of
>#if ENABLE_FEATURE_DEVFSD_VERBOSE || ENABLE_DEBUG
>etc..



More information about the busybox mailing list