[U-Boot] [PATCH v0 0/4] env: reworking + default/import individual vars

Gerlando Falauto gerlando.falauto at keymile.com
Tue Nov 8 13:04:13 CET 2011


On 11/08/2011 12:46 PM, Wolfgang Denk wrote:
> Dear Gerlando Falauto,
>
> In message<4EB8F762.1030508 at keymile.com>  you wrote:
>>
>> I tested "env import" with and without -n. Same for "env default".
>> Also tested special variables.
>>
>>   >  For me, env import is broken now.
>>
>> I am not able to see how it's obviously broken.
>
> It just didn't work for me.

Honestly, I didn't try it against the latest master branch.

>> Perhaps you mean that when you don't provide an argument to -n, the
>> following argument (e.g., -t) is interpreted as a variable name (and
>> therefore consumed)? That doesn't look like an error to me.
>
> Yes, this is a serious bug. Aor design problem.
>
> The whole interface is wrong. I have just extendend "env export" to
> do a similar thing. The new syntax is:
>
>           env export [-t | -b | -c] [-s size] addr [var ...]

When/where? I can't see that in the latest master branch.

> We should do exactly the same for "env
> import", i. e. make the size parameter an option to be passed with
> "-s size", and pass the names of any variables to import as additional
> arguments:
>
> 	env import [-t | -b | -c] [-s size] addr [var ...]
>
> This also greatly simplifies the implementation.

Totally agree. Question is: won't that break all the existing scripts???

If you agree, I can post an updated patch with the same behavior for 
import. Although I am bit concerned about our scripts.

Best,
Gerlando Falauto


More information about the U-Boot mailing list