[PATCH 0/3] add "call" command
Rasmus Villemoes
rasmus.villemoes at prevas.dk
Fri Sep 25 15:38:16 CEST 2020
On 25/09/2020 15.09, Wolfgang Denk wrote:
> Dear Heinrich Schuchardt,
>
> In message <4b00225d-d960-4a14-9aec-110ddddf7f30 at gmx.de> you wrote:
>>
>> Further we cannot first introduce a command call and then eliminate it
>> due to backward compatibility. We should decide on the final version
>> beforehand.
Please note that I never meant for
f a b c
to be an eventual replacement for
call f a b c
the latter syntax would continue to be accepted. And I'm personally
completely fine with that being the _only_ way to call a
function-defined-in-the-environment-with-positional-args, which also makes
>> I am
>> not sure if a built in command should take precedence over a variable of
>> the same name or the other way round.
a moot point.
So can we instead discuss whether the "call" command is worth having at
all. I notice that Wolfgang calls this a nice idea (thanks), but
soft-NAKs it because he'd rather see all of hush updated before adding
extra features. The thing is, I can't take that monumental task on me
(especially with all the backwards-compatibility concerns there'd be,
with various scripts in the wild that may have come to rely on U-Boot's
hush parser's behaviour in corner cases), but doing cmd_call is about
100 lines of mostly stand-alone code.
Rasmus
More information about the U-Boot
mailing list