From 4aa8b3a351fb6ac6b5d56d30dbda0d44e324a806 Mon Sep 17 00:00:00 2001 From: Joshua Lock Date: Wed, 23 May 2012 13:50:38 -0700 Subject: [PATCH] Revert "sanity.bbclass: check user can read and write to SSTATE_DIR" This has caused problems for several users, including the Yocto Project autobuilder. Since the message was added in order to be more user friendly revert the change. This reverts commit 0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c. Signed-off-by: Richard Purdie --- meta/classes/sanity.bbclass | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass index ff3c413..05545f4 100644 --- a/meta/classes/sanity.bbclass +++ b/meta/classes/sanity.bbclass @@ -96,16 +96,10 @@ 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 != "": - # 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") - + testmsg = check_create_long_filename(sstate_dir, "SSTATE_DIR") return testmsg def check_sanity_tmpdir_change(tmpdir, data): @@ -156,12 +150,7 @@ 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\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 "Failed to create a file in %s: %s" % (pathname, strerror) return "" def check_connectivity(d): -- 2.7.4