diff options
author | Chris Laplante <chris.laplante@agilent.com> | 2021-06-02 12:20:38 -0400 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-06-06 23:18:26 +0100 |
commit | 66f22c0bd19a81ee7ccb49543f24a4f90fc012ba (patch) | |
tree | 8c24b8d7caafe588a9692c90cf6e1368dc229567 /bitbake | |
parent | c9ef89b3b9730bcdaea29af095bd88b6f1c5a6db (diff) | |
download | poky-66f22c0bd19a81ee7ccb49543f24a4f90fc012ba.tar.gz |
bitbake: build: warn on setting noexec/nostamp/fakeroot flag to any value besides '1'
Currently setting those flags to even the empty string "" causes it to be set,
which is contrary to the documentation. In a future version of BitBake, we'd
like to change the behavior so that setting the flag to "" does not set it.
This will allow conditionally setting noexec, using variable expansion or
inline Python.
I found no places in poky or meta-openembedded where this warning would trigger.
[YOCTO #13808]
(Bitbake rev: 1e7655c4f765ba7b4791c4cca048a69bf8d9c93d)
Signed-off-by: Chris Laplante <chris.laplante@agilent.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake')
-rw-r--r-- | bitbake/lib/bb/build.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/bitbake/lib/bb/build.py b/bitbake/lib/bb/build.py index b2715fc530..6ce8f1e6d3 100644 --- a/bitbake/lib/bb/build.py +++ b/bitbake/lib/bb/build.py | |||
@@ -927,6 +927,11 @@ def add_tasks(tasklist, d): | |||
927 | task_deps[name] = {} | 927 | task_deps[name] = {} |
928 | if name in flags: | 928 | if name in flags: |
929 | deptask = d.expand(flags[name]) | 929 | deptask = d.expand(flags[name]) |
930 | if name in ['noexec', 'fakeroot', 'nostamp']: | ||
931 | if deptask != '1': | ||
932 | bb.warn("In a future version of BitBake, setting the '{}' flag to something other than '1' " | ||
933 | "will result in the flag not being set. See YP bug #13808.".format(name)) | ||
934 | |||
930 | task_deps[name][task] = deptask | 935 | task_deps[name][task] = deptask |
931 | getTask('mcdepends') | 936 | getTask('mcdepends') |
932 | getTask('depends') | 937 | getTask('depends') |