Re: [RFC PATCH 02/28] cli: Add LIL shell

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1599 bytes --]

On Tue, Jul 06, 2021 at 08:57:23AM -0600, Simon Glass wrote:
> Hi Michael,
> 
> On Tue, 6 Jul 2021 at 01:53, Michael Nazzareno Trimarchi
> <michael@amarulasolutions.com> wrote:
> >
> > On Tue, Jul 6, 2021 at 9:46 AM Wolfgang Denk <wd@denx.de> wrote:
> > >
> > > Dear Sean,
> > >
> > > In message <e7e23702-4e55-f785-c77f-2d3076fdd67b@gmail.com> you wrote:
> > > >
> > > > >>> foo() {                           proc foo {first second} {
> > > > >>>   echo $1 $2                      echo $first $second
> > > > >>> }                         }
> > > >
> > > > This is not possible. We only have eval (run) as of today. I view adding
> > > > functions as one of the most important usability improvements we can
> > > > make.
> > >
> > > Again:  this is not an issue with hush as is, but only with our
> > > resource-limited port of a nearly 20 year old version of it.
> > >
> > > Updating to a current version would fix this, in an almost 100%
> > > backward compatible way.
> >
> > Agree on this. There is another email about the LIL and how is right now
> > maintained and if the solutions that we have already in place can be just
> > updated, I don't find any valid reason to change it but update
> > seems more straightforward.
> 
> Would you be interested in taking this on? It has been talked about
> for some years...

To repeat myself, someone posting the patches to update (or more likely,
replace as a new Kconfig option and I'd be fine with a "pick old or new"
as a starting point) is the main gating factor on using a new version of
hush.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]