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

Rob Landley rob at landley.net
Fri May 12 18:20:49 UTC 2006


On Friday 05 May 2006 9:35 pm, Shaun Jackman wrote:
> Ping! Any comments?
>
> Cheers,
> Shaun

Top posting...

> On 4/21/06, Shaun Jackman <sjackman at gmail.com> wrote:
> > This patch allows setting an environment variable without an explicit
> > `export' statement. I understand there's a moratorium of features on
> > lash, but I consider 'foo=bar' a fairly basic and necessary syntax. On
> > my ARM7 system this patch adds 32 bytes to the .text section.

Hmmm...  If adding anything at all to lash is reasonable, this probably would 
be.  It supports "this=that" as a statement, which would ordinarily set a 
shell internal variable except lash doesn't have any.  It doesn't support 
"this=that command"...

I'm trying to figure out if it is.  Lash is already a funky custom environment 
to program against...

Very definitely on the fence here...

Rob
-- 
Never bet against the cheap plastic solution.



More information about the busybox mailing list