On Monday, September 05, 2011 04:27:28 PM Wolfgang Denk wrote:
> Dear Marek Vasut,
>
> In message <201109051512.48148.marek.va...@gmail.com> you wrote:
> > > + setenv("filesize", buf);
> >
> > maybe you want to check set_local_var() too ?
>
> Yes, we should clean up all these auto-generated environment variables
> one day. But what you propose here is not a good approach:
>
> (1) not all boards use the hush shell, and set_local_var() is only
> available there.
> (2) if you make such a change, then you must make it everywhere, and
> you must make sure not to break existent use of these things.
>
> Both is definitely out of the scope of this patch.
You're definitelly right on this thing.
>
> Best regards,
>
> Wolfgang Denk
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
