[U-Boot-Users] [PATCH] Propagate Error Status to the Shell on fw_printenv Errors

Grant Erickson gerickson at nuovations.com
Wed May 7 15:53:23 CEST 2008


On 5/7/08 12:32 AM, Markus Klotzbücher wrote:
> Wolfgang Denk <wd at denx.de> writes:
>> In message <87myn2ehqm.fsf at denx.de> you wrote:
> Oh right, yes. Shouldn't post to lists before drinking coffee in the
> morning.
> 
>> However, the addition of this feature is undocumented (not mentioned
>> in the commit message), and I don;t see why we should restrict it to a
>> single variable - it may be useful to print several values here, too,
>> for example like that:
>> 
>> $ set `fw_printenv -n netdev ipaddr netmask`
>> $ ifconfig $1 $2 netmask $3
>> 
>> [Well, I know that's a bad example because  it's  missing  all  error
>> checking, but you get the idea.]
> 
> Yes, that would nice indeed. Grant, care to add this? And please update
> the documentation in any case.

Markus,

Thanks for the comments. As suggested by Wolfgang, the "-n" option was
pre-existing in the code before my patch.

I can investigate extending "-n" as described above; however, the caveat is
that the user knows that the values of some variables may have spaces. In
such a case, he/she might find the above example to not work as expected.

Anyway, hopefully Wolfgang can ACK the patch as is and I can look at
extending the "-n" option as another separate effort.

Regards,

Grant

PS: To which documentation are you referring? The comments in the source or
the DULG?






More information about the U-Boot mailing list