[PATCH v2 01/38] Makefile: Fake external blobs by default with binman

Rasmus Villemoes rasmus.villemoes at prevas.dk
Mon Oct 10 12:15:58 CEST 2022


On 10/01/2022 04.13, Simon Glass wrote:
> This behaviour is necessary with boards where the binman description
> requires processing external blobs, since these may be missing.
> 
> Enable it by default, so that CI is happy. Warnings indicate that a valid
> image is not produced, as with the --allow-missing option.

I know I have talked and warned about this before, and now I was
actually bitten by it IRL.

Can we _please_ stop doing this by default. I understand why upstream
U-Boot's CI system needs this, but it should be possible for that CI
system to set an environment variable or pass a make parameter to opt-in
to generating these fake blobs.

Generating potentially non-functional binaries by default, and currently
without even a way to opt out of that (that I have found), is frankly
not sane. Yeah, a warning is printed, but that easily scrolls away or in
the case of automated builds is hidden away in some log file people
would only ever look at if the job failed.

I want my/our CI to _fail hard_ when I have failed to update the Yocto
metadata to stage the necessary blobs before do_compile. And if upstream
U-Boot want to continue to have this by default, can we at the very
least get an ergonomic way to opt out (preferably a CONFIG_ option I can
set or clear in my out-of-tree defconfig files).

Rasmus


More information about the U-Boot mailing list