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

Wolfgang Denk wd at denx.de
Fri Oct 15 16:32:25 CEST 2021


Dear Simon Glass,

In message <20211014122254.v6.4.Ie78bfbfca0d01d9cba501e127f446ec48e1f7afe at changeid> you wrote:
> At present U-Boot environment variables, and thus scripts, are defined
> by CONFIG_EXTRA_ENV_SETTINGS. It is painful to add large amounts of text
> to this file and dealing with quoting and newlines is harder than it
> should be. It would be better if we could just type the script into a
> text file and have it included by U-Boot.
>
> Add a feature that brings in a .env file associated with the board
> config, if present. To use it, create a file in a board/<vendor>/env
> directory called <board>.env (or common.env if you want the same
> environment for all boards).

Argh... did you bother to read my comments?  Apparently not.

Thus:

NAKed by: Wolfgang Denk <wd at denx.de>


I really think your fixed filename proposal does not work well in
reality.  The file name should be Kconfig configurable. See [1]
for details.

[1] https://lists.denx.de/pipermail/u-boot/2021-October/462668.html


Thanks.

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
"There's always been Tower of Babel sort of  bickering  inside  Unix,
but  this  is the most extreme form ever. This means at least several
years of confusion." - Bill Gates, founder and chairman of Microsoft,
about the Open Systems Foundation


More information about the U-Boot mailing list