diff options
author | Ross Burton <ross.burton@intel.com> | 2017-09-20 13:57:33 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-09-22 17:15:29 +0100 |
commit | 8e1287867fdb27fc6ff11787636fafe600ff2b0b (patch) | |
tree | 5ffc82dba3dec6dd6899f60fd6069a9fdf869092 | |
parent | 4b560c98345ebcbcb67fc032af08781f92e0454e (diff) | |
download | poky-8e1287867fdb27fc6ff11787636fafe600ff2b0b.tar.gz |
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.
(From OE-Core rev: 2fb74abbe07b6b82a715ac0fe16449bd8420110e)
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/classes/sanity.bbclass | 5 |
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): | |||
726 | if not ( check_conf_exists("conf/distro/${DISTRO}.conf", d) or check_conf_exists("conf/distro/include/${DISTRO}.inc", d) ): | 726 | if not ( check_conf_exists("conf/distro/${DISTRO}.conf", d) or check_conf_exists("conf/distro/include/${DISTRO}.inc", d) ): |
727 | status.addresult("DISTRO '%s' not found. Please set a valid DISTRO in your local.conf\n" % d.getVar("DISTRO")) | 727 | status.addresult("DISTRO '%s' not found. Please set a valid DISTRO in your local.conf\n" % d.getVar("DISTRO")) |
728 | 728 | ||
729 | # Check that these variables don't use tilde-expansion as we don't do that | ||
730 | for v in ("TMPDIR", "DL_DIR", "SSTATE_DIR"): | ||
731 | if d.getVar(v).startswith("~"): | ||
732 | status.addresult("%s uses ~ but Bitbake will not expand this, use an absolute path or variables." % v) | ||
733 | |||
729 | # Check that DL_DIR is set, exists and is writable. In theory, we should never even hit the check if DL_DIR isn't | 734 | # Check that DL_DIR is set, exists and is writable. In theory, we should never even hit the check if DL_DIR isn't |
730 | # set, since so much relies on it being set. | 735 | # set, since so much relies on it being set. |
731 | dldir = d.getVar('DL_DIR') | 736 | dldir = d.getVar('DL_DIR') |