diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-04-12 19:54:20 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-04-13 10:54:10 +0100 |
commit | ffabe5d1b9539c6cc9fc87ecff9d40f69f8df9be (patch) | |
tree | 62ea4ed5fb0e62650be2697448bb9c0a30d93e8b /scripts | |
parent | 3f32ca2a572cc87a1f25c78ab89bf740c19db088 (diff) | |
download | poky-ffabe5d1b9539c6cc9fc87ecff9d40f69f8df9be.tar.gz |
oe-selftest: Error if known problem variables are set
Setting SANITY_TESTED_DISTROS or PRSERV_HOST are known to break oe-selftest.
Rather than have the user experience this, refuse to execute unless the
environment is correct.
Ideally we'd try and unset these but that is a more invasive change and this
at least makes people aware of the problem.
[YOCTO #11292]
(From OE-Core rev: 0c9b981e88c76da316e76f17e6da3a03b87c5008)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/oe-selftest | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/scripts/oe-selftest b/scripts/oe-selftest index 58d3b1ef8a..52366b1c8d 100755 --- a/scripts/oe-selftest +++ b/scripts/oe-selftest | |||
@@ -146,6 +146,14 @@ def preflight_check(): | |||
146 | log.error("You have buildhistory enabled already and this isn't recommended for selftest, please disable it first.") | 146 | log.error("You have buildhistory enabled already and this isn't recommended for selftest, please disable it first.") |
147 | return False | 147 | return False |
148 | 148 | ||
149 | if get_bb_var("PRSERV_HOST"): | ||
150 | log.error("Please unset PRSERV_HOST in order to run oe-selftest") | ||
151 | return False | ||
152 | |||
153 | if get_bb_var("SANITY_TESTED_DISTROS"): | ||
154 | log.error("Please unset SANITY_TESTED_DISTROS in order to run oe-selftest") | ||
155 | return False | ||
156 | |||
149 | log.info("Running bitbake -p") | 157 | log.info("Running bitbake -p") |
150 | runCmd("bitbake -p") | 158 | runCmd("bitbake -p") |
151 | 159 | ||