[patch] crond misc

[Rob Landley]

On Sunday 14 May 2006 8:45 am, Bernhard Fischer wrote:
> 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.

You know what?  You could do this.  I'm not spending my time doing this right 
now.  If I'm supposed to be spending more time testing fewer changes, 
spending time on this sort of thing would be a luxury I can't afford.  
Therefore, you're asking me to spend my time on something you don't actually 
want me to spend time on.

You do it.

Rob
-- 
Never bet against the cheap plastic solution.