[PATCH v6 4/7] env: Allow U-Boot scripts to be placed in a .env file

Wolfgang Denk wd at denx.de
Mon Oct 18 16:10:34 CEST 2021


Dear Tom,

In message <20211018133728.GQ7964 at bill-the-cat> you wrote:
> 
> > And please see also my comments about changing the autostart
> > functionality for the user.
>
> Perhaps we should just make "+" an illegal character in the variable
> name, for consistency?

And break backward compatibility?  I'd rather see a better
definition of the syntax of the environment files, plus maybe a more
powerful parser.

I mean, there is no technical reason to forbid the '+' character -
and then it's only at the end of the variable name.

Hm... I can't find it right now but did I not also read about other
restrictions to variable names, like they must noch begin with '_'
when using this new tool?


I feel it is wrong to place new restrictions on something that was
constant for 21 years, just because our parser cannot parse it...

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"You ain't experienced..." "Well, nor are you." "That's true. But the
point is ... the point is ... the point is we've been not experienced
for a lot longer than you. We've got  a  lot  of  experience  of  not
having any experience."           - Terry Pratchett, _Witches Abroad_


More information about the U-Boot mailing list