]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core-contrib.git/commitdiff
bitbake.conf: set BB_DEFAULT_UMASK using ??=
authorRasmus Villemoes <rasmus.villemoes@prevas.dk>
Fri, 26 Aug 2022 12:20:44 +0000 (14:20 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 30 Aug 2022 09:28:26 +0000 (10:28 +0100)
Currently, there's no way for the user's site.conf, local.conf or
similar to set BB_DEFAULT_UMASK, because those files are included by
bitbake.conf prior to the unconditional assignment of
BB_DEFAULT_UMASK. To make that possible, use a weak default assignment
instead. This is also consistent with most other variable assignments
in the lower half of bitbake.conf.

I believe the risk of a regression is very small; it would require
something like somebody having a definition of BB_DEFAULT_UMASK in a
local configuration file, and having been relying on that _not_ taking
effect.

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
meta/conf/bitbake.conf

index dd2df8a5520cdb2d928af2ccdc1244e3a8dcd996..52a36d788bf9da3f03ffc85ce9fd62ae86b63c4b 100644 (file)
@@ -928,7 +928,7 @@ SHELL[unexport] = "1"
 TRANSLATED_TARGET_ARCH ??= "${@d.getVar('TARGET_ARCH').replace("_", "-")}"
 
 # Set a default umask to use for tasks for determinism
-BB_DEFAULT_UMASK = "022"
+BB_DEFAULT_UMASK ??= "022"
 
 # Complete output from bitbake
 BB_CONSOLELOG ?= "${LOG_DIR}/cooker/${MACHINE}/${DATETIME}.log"