[U-Boot] [PATCH 1/2] env: set individual variables to default

Wolfgang Denk wd at denx.de
Wed Sep 28 23:08:09 CEST 2011


Dear Gerlando Falauto,

In message <4E82E6CA.9030802 at keymile.com> you wrote:
>
> > env import -n name[,..] [other_flags] addr [size]
> >
> > ?
> 
> Uhm, wouldn't that make the syntax completely unrelated to all other 
> commands, leading to confusion?
> 
> How about something like:
> 
> env import [-f] [flags] addr [size] [-n name[ ...]]
> env default [-f] -a|-n name[ ...]
> env set [-f] name [val ...]

No.  That's even worse.  If you don't like the comma separation,we
could as well accept multiple -n args:

env import -n name [-n name1 ... ] [other_flags] addr [size]

> Where:
>   -a in "env default" would be the way to prevent the inadverent user
>      from wrecking the environment by mistake.

What does -a stand for?  "all" ?

> Slightly off-topic: how about variables starting with "-"?

Should be no problem in general, except for the pathological cases
like variable names "-f" etc.   But there has to be a penalty for such
stupid names :-)

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Insufficient facts always invite danger.
	-- Spock, "Space Seed", stardate 3141.9


More information about the U-Boot mailing list