include: add umask prereq check
authorJo-Philipp Wich <jo@mein.io>
Tue, 4 Oct 2016 09:04:50 +0000 (11:04 +0200)
committerJo-Philipp Wich <jo@mein.io>
Tue, 4 Oct 2016 09:08:58 +0000 (11:08 +0200)
When building LEDE with umask values other than 022, the resulting packages
will embed improper permissions, which may lead to random errors or non-
functional scripts on the target.

In order to make users aware of this problem, add a build-prereq check to
assert a correct umask setting before starting the build.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
include/prereq-build.mk

index 9c0717d53a8beaab2cec5958fbf981d8796a17b1..c3b88db6271631e6fb5a8f720825fcb547dd7d67 100644 (file)
@@ -24,6 +24,10 @@ $(eval $(call TestHostCommand,case-sensitive-fs, \
        rm -f $(TMP_DIR)/test.*; touch $(TMP_DIR)/test.fs; \
                test ! -f $(TMP_DIR)/test.FS))
 
+$(eval $(call TestHostCommand,proper-umask, \
+       Please build with umask 022 - other values produce broken packages, \
+       umask | grep -xF 0022))
+
 $(eval $(call SetupHostCommand,gcc, \
        Please install the GNU C Compiler (gcc), \
        $(CC) --version | grep gcc, \