[poky] [PATCH 3/3] sanity.bbclass: add check for creation of long filenames
Paul Eggleton
paul.eggleton at linux.intel.com
Thu Dec 16 09:35:31 PST 2010
Detect and fail if filesystem in use for TMPDIR or SSTATE_DIR has an
unreasonably short file name length limit (eg. eCryptFS). This can cause
"file name too long" errors during poky builds (e.g. when writing sstate
files for packages with a git revision as the version).
Signed-off-by: Paul Eggleton <paul.eggleton at linux.intel.com>
---
meta/classes/sanity.bbclass | 28 ++++++++++++++++++++++++++--
1 files changed, 26 insertions(+), 2 deletions(-)
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 969cc2e..fce0cbd 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -23,16 +23,40 @@ def check_conf_exists(fn, data):
def check_sanity_sstate_dir_change():
# Sanity checks to be done when the value of SSTATE_DIR changes
- return ""
+ # Check that SSTATE_DIR isn't on a filesystem with limited filename length (eg. eCryptFS)
+ #
+ testmsg = ""
+ sstatedir = data.getVar('SSTATE_DIR', e.data, True) or ''
+ if sstatedir != "":
+ testmsg = check_create_long_filename(sstatedir, "SSTATE_DIR")
+ return testmsg
def check_sanity_tmpdir_change():
# Sanity checks to be done when the value of TMPDIR changes
- return ""
+
+ # Check that TMPDIR isn't on a filesystem with limited filename length (eg. eCryptFS)
+ testmsg = check_create_long_filename(tmpdir, "TMPDIR")
+ return testmsg
def check_sanity_version_change():
# Sanity checks to be done when SANITY_VERSION changes
return ""
+def check_create_long_filename(filepath, pathname):
+ testfile = os.path.join(filepath, ''.join([`num`[-1] for num in xrange(1,200)]))
+ try:
+ if not os.path.exists(filepath):
+ bb.utils.mkdirhier(filepath)
+ f = file(testfile, "w")
+ f.close()
+ os.remove(testfile)
+ except IOError as (errno, strerror):
+ 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 ""
+
def check_sanity(e):
from bb import note, error, data, __version__
--
1.7.1
More information about the poky
mailing list