[PATCH v2 2/3] allow positional arguments with "run" command

Simon Glass
2020-10-15T15:05:31Z

Hi Rasmus,

On Mon, 12 Oct 2020 at 01:06, Rasmus Villemoes
<rasmus.villemoes@prevas.dk> wrote:
>
> On 12/10/2020 05.34, Simon Glass wrote:
> > On Wed, 7 Oct 2020 at 01:21, Rasmus Villemoes
> > <rasmus.villemoes@prevas.dk> wrote:
> >>
> >> cmd/Kconfig | 10 ++++++++++
> >> cmd/nvedit.c | 7 ++++++-
> >> common/cli.c | 44 ++++++++++++++++++++++++++++++++++++++------
> >> common/cli_hush.c | 32 +++++++++++++++++++++++++++++++-
> >> include/cli_hush.h | 9 +++++++++
> >> 5 files changed, 94 insertions(+), 8 deletions(-)
> >>
> >
> > Reviewed-by: Simon Glass <sjg@chromium.org>
> >
> > I'm not sure where the previous discussion went. But please think
> > about how we can add some tests here.
>
> Isn't that exactly what I do in 3/3? Or are you thinking of something else?
>

Yes that's good, but is the plan now to take these patches rather than
update to the latest hush? I was wondering is Buzybox has any tests
for hush.

Regards,
Simon