Re: [BusyBox] builtin echo for ash

[Peter Mazinger]

>  > > one other implementation question:  moving the echo code into
>  > > libbb/bb_echo.c now requires that CONFIG_FANCY_ECHO be accessible
>  > > from somewhere other than just the coreutils config menu, since
>  > > you might want to configure the builtin echo without configuring
>  > > the echo applet.  i wasn't sure how to handle this, so i simply
>  > > duplicated the CONFIG_FANCY_ECHO option into the ash config menu.
>  > > this isn't right -- what should i do instead?
>  > 
>  > i dont think the duplication will be that big of a deal ... but i dont
> know 
>  > kconfig will handle it :)
> 
> busybox menuconfig handles it surprisingly well -- not only does
> it only emit one instance of the option, but it's impossible to
> set the two instances of the option to different values.  whether
> it was intentional or not, it's exactly the right behavior.  i just
> don't like duplicating the menu text in two places.

you could create a config option (ex. CONFIG_BB_ECHO) that is not selectable
by menuconfig and use select CONFIG_BB_ECHO for the 2 cases you need it

Peter

-- 
5 GB Mailbox, 50 FreeSMS http://www.gmx.net/de/go/promail
+++ GMX - die erste Adresse für Mail, Message, More +++