Dear Stefano,

In message <[EMAIL PROTECTED]> you wrote:
> The command interpreter checks always if an environment variable
> for that name exists and in this case the content of the variable
> is executed. It becomes possible to redefine all U-Boot commands.
> A new "builtin" command is added to be able to run builtin U-boot commands
> even if they are redefined.

Cool. That's just great.

If I had known it's suffcient to put ideas on  a  web  page  to  have
people  put  them  into  code  I  would  have started doing this much
earlier :-)

> +                     /* Check if exists a variable with that name */
> +                     if ((p = getenv (child->argv[i])) != NULL )  {
> +                             int rcode;
> +                             rcode = (parse_string_outer(p,
> +                                         FLAG_PARSE_SEMICOLON | 
> FLAG_EXIT_FROM_LOOP) != 0);
> +                             return rcode;
> +                     }

I think this covers only the hush shell version. We need something
like this instead:


#ifdef CFG_HUSH_PARSER
                rcode = parse_string_outer(p, FLAG_PARSE_SEMICOLON |
                                    FLAG_EXIT_FROM_LOOP);
#else 
                rcode =run_command (p, 0);
#endif


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
For every complex problem, there is a solution that is simple,  neat,
and wrong.                                               - Mark Twain
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to