summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoshua Lock <josh@linux.intel.com>2012-05-11 18:00:26 -0700
committerRichard Purdie <richard.purdie@linuxfoundation.org>2012-05-17 21:11:26 +0100
commit0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c (patch)
tree4766867b9e0a6fa0e075e1191319aea592788c6d
parentafa2599f3c2d0428714f947814a25db4ce245caf (diff)
downloadopenembedded-core-0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c.tar.gz
openembedded-core-0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c.tar.bz2
openembedded-core-0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c.tar.xz
openembedded-core-0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c.zip
sanity.bbclass: check user can read and write to SSTATE_DIR
The user needs read and write permissions to SSTATE_DIR, check whether they have sufficient permissions and if not recommend use of SSTATE_MIRRORS. Signed-off-by: Joshua Lock <josh@linux.intel.com>
-rw-r--r--meta/classes/sanity.bbclass17
1 files changed, 14 insertions, 3 deletions
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 05545f45f..ff3c41301 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -96,10 +96,16 @@ def check_conf_exists(fn, data):
def check_sanity_sstate_dir_change(sstate_dir, data):
# Sanity checks to be done when the value of SSTATE_DIR changes
- # Check that SSTATE_DIR isn't on a filesystem with limited filename length (eg. eCryptFS)
testmsg = ""
if sstate_dir != "":
- testmsg = check_create_long_filename(sstate_dir, "SSTATE_DIR")
+ # Check that the user can read and write to SSTATE_DIR
+ sstatemsg = check_can_read_write_directory(sstate_dir) or None
+ if sstatemsg:
+ sstatemsg = sstatemsg + ". You could try using it as an SSTATE_MIRRORS instead of SSTATE_CACHE.\n"
+ testmsg = testmsg + sstatemsg
+ # Check that SSTATE_DIR isn't on a filesystem with limited filename length (eg. eCryptFS)
+ testmsg = testmsg + check_create_long_filename(sstate_dir, "SSTATE_DIR")
+
return testmsg
def check_sanity_tmpdir_change(tmpdir, data):
@@ -150,7 +156,12 @@ def check_create_long_filename(filepath, pathname):
if errno == 36: # ENAMETOOLONG
return "Failed to create a file with a long name in %s. Please use a filesystem that does not unreasonably limit filename length.\n" % pathname
else:
- return "Failed to create a file in %s: %s" % (pathname, strerror)
+ return "Failed to create a file in %s: %s\n" % (pathname, strerror)
+ return ""
+
+def check_can_read_write_directory(directory):
+ if not os.access(directory, os.R_OK|os.W_OK):
+ return "Insufficient permissions for %s" % directory
return ""
def check_connectivity(d):