[patch] crond misc

Bernhard Fischer rep.nop at aon.at
Sun May 14 12:45:09 UTC 2006


On Thu, May 11, 2006 at 11:15:12AM -0400, Rob Landley wrote:
>On Wednesday 10 May 2006 1:10 pm, Bernhard Fischer wrote:
>> >Looking at crond.c, I notice that it's got lots of #ifdefs for
>> >FEATURE_DEBUG_OPT, which can never be set by our config system.  Any idea
>> >what that's for?  (It should either be added to the config system or
>> > removed entirely.  I can't clean it up to use ENABLE and USE macros
>> > unless it's in the config system...)
>>
>> This should be a CONFIG_FEATURE_CROND_DEBUG, yes.
>
>The downside of doing that is it would be enabled by make defconfig.  Should 
>it be?

No, it shouldn't, so it ought to be a CONFIG_DEBUG_CROND or the like.
>
>CONFIG_FEATURE_X is enabled by defconfig.  CONFIG_DEBUG_X isn't.



More information about the busybox mailing list