[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