[PATCH] shell/lash.c: set environment variable

Shaun Jackman sjackman at gmail.com
Fri May 26 16:11:03 UTC 2006


On 5/26/06, Rob Landley <rob at landley.net> wrote:
...
> I'm pondering trying to refactor lash into the base for bbsh.  I'd have
> already started if the lash code wasn't full of warnings that this is not
> recommended...

When was the last survey of the shell phylogeny undertaken? Perhaps we
can find a better starting point than lash. Debian, for example, uses
dash -- the Debian Almquist shell -- on its installation media.

$ size /bin/dash
   text    data     bss     dec     hex filename
  79132     892   10240   90264   16098 /bin/dash

If dash were to make use of libbb, and peppered liberally with
CONFIG_FEATURE_DASH_xxx, I could see the result being a very standard,
configurable, small shell.

...
> I suppose in the short term I could always add this as a config option, even
> though we're talking something like 35 bytes here...

For the 35 bytes saved, I wouldn't bother with the config option. I
don't know where exactly one places the cutoff, but for me it's
somewhere above 35 bytes -- and for context, I'm on a 1 MB RAM & 2 MB
flash system.

Cheers,
Shaun



More information about the busybox mailing list