[PATCH v3 1/2] imagebuilder: unset BINARY_FOLDER and DOWNLOAD_FOLDER in final archive
Sven Roederer
devel-sven at geroedel.de
Thu May 6 20:40:32 BST 2021
Baptiste,
Am Donnerstag, 6. Mai 2021, 13:13:46 CEST schrieb Baptiste Jonglez:
> >
> > + $(SED) 's/^CONFIG_BINARY_FOLDER=.*/# CONFIG_BINARY_FOLDER was reset by
> > Imagebuilder/' $(PKG_BUILD_DIR)/.config + $(SED)
> > 's/^CONFIG_DOWNLOAD_FOLDER=.*/# CONFIG_DOWNLOAD_FOLDER was reset by
> > Imagebuilder/' $(PKG_BUILD_DIR)/.config
> Several scripts assume that this exact format is used for disabled config:
> "# CONFIG_FOO is not set" (same as in the kernel).
>
> While your patch works [probably because these scripts are not run when
> using the ImageBuilder], it seems cleaner to stick to the standard format,
> e.g. in case a .config file is copied around from an imagebuilder.
>
The intention to not use th normal "unset" message was to leave a pointer why
it's unset in contrast to the build-time file, in case someone relies on it.
But I'm happy to us the normal message here.
Sven
More information about the openwrt-devel
mailing list