Re: [PATCH v7 5/7] doc: Mention CONFIG_DEFAULT_ENV_FILE

Wolfgang Denk
2021-10-18T13:56:20Z

Dear Simon,

In message <20211016003339.723169-4-sjg@chromium.org> you wrote:
> Add mention of this option this it does a similar thing to the text
> environment.
>
> Suggested-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
...

> +The format is the same as accepted by the mkenvimage tool: lines containing
> +key=value pairs, blank lines and lines beginning with # are ignored.

This is misleading. It can be read as: "lines containing key=value
pairs ... are ignored."

This reminds me of a sign in the door of a shop in Cornwall which
read: "No prams or dogs or food being eaten!" I mean eating dogs is
bad enough, but eating prams? ? ?

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@denx.de
"There's only one way to have a happy marriage and as soon as I learn
what it is I'll get married again." - Clint Eastwood