summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@intel.com>2017-09-20 13:57:33 +0100
committerRichard Purdie <richard.purdie@linuxfoundation.org>2017-09-22 17:14:42 +0100
commit2fb74abbe07b6b82a715ac0fe16449bd8420110e (patch)
tree9274f4bb314ab03e4aee1a56bc8bd32b6c584c3e
parentb12d99dbfbee8c4b3680f453f833410950238bb9 (diff)
downloadopenembedded-core-contrib-2fb74abbe07b6b82a715ac0fe16449bd8420110e.tar.gz
openembedded-core-contrib-2fb74abbe07b6b82a715ac0fe16449bd8420110e.tar.bz2
openembedded-core-contrib-2fb74abbe07b6b82a715ac0fe16449bd8420110e.zip
sanity: check that path variables don't use ~
The core path variables (TMPDIR, DL_DIR, SSTATE_DIR) don't use tilde expansion but if the user does then the errors are very mysterious, so check on startup. Signed-off-by: Ross Burton <ross.burton@intel.com>
-rw-r--r--meta/classes/sanity.bbclass5
1 files changed, 5 insertions, 0 deletions
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 91b92ec28a..1feb7949da 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -726,6 +726,11 @@ def check_sanity_everybuild(status, d):
if not ( check_conf_exists("conf/distro/${DISTRO}.conf", d) or check_conf_exists("conf/distro/include/${DISTRO}.inc", d) ):
status.addresult("DISTRO '%s' not found. Please set a valid DISTRO in your local.conf\n" % d.getVar("DISTRO"))
+ # Check that these variables don't use tilde-expansion as we don't do that
+ for v in ("TMPDIR", "DL_DIR", "SSTATE_DIR"):
+ if d.getVar(v).startswith("~"):
+ status.addresult("%s uses ~ but Bitbake will not expand this, use an absolute path or variables." % v)
+
# Check that DL_DIR is set, exists and is writable. In theory, we should never even hit the check if DL_DIR isn't
# set, since so much relies on it being set.
dldir = d.getVar('DL_DIR')