From 967bc92b0b4daec99210ebca7ef0b896ef22de38 Mon Sep 17 00:00:00 2001 From: Joshua Lock Date: Mon, 5 Sep 2016 09:20:24 +0100 Subject: oe.path: fix copyhardlinktree() The change to preserve extended attributes in copytree() and copyhardlinktree() (e591d69103a40ec4f76d1132a6039d9cb1555103) resulted in an incorrect cp invocation in copyhardlinktree() when the source directory contained hidden files. This was because the passed src was modified in place but some code paths expected it to remain unmodified from the passed value. Resolve the issue by constructing a new source string, rather than modifying the passed in string. Signed-off-by: Joshua Lock --- meta/lib/oe/path.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/meta/lib/oe/path.py b/meta/lib/oe/path.py index 631c3b430c..06a5af2659 100644 --- a/meta/lib/oe/path.py +++ b/meta/lib/oe/path.py @@ -79,12 +79,15 @@ def copyhardlinktree(src, dst): # writers try and create a directory at the same time cmd = "cd %s; find . -type d -print | tar --xattrs --xattrs-include='*' -cf - -C %s -p --no-recursion --files-from - | tar --xattrs --xattrs-include='*' -xf - -C %s" % (src, src, dst) subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) + source = '' if os.path.isdir(src): import glob if len(glob.glob('%s/.??*' % src)) > 0: - src = src + '/.??* ' - src = src + '/*' - cmd = 'cp -afl --preserve=xattr %s %s' % (src, dst) + source = '%s/.??* ' % src + source = source + '%s/*' % src + else: + source = src + cmd = 'cp -afl --preserve=xattr %s %s' % (source, dst) subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) else: copytree(src, dst) -- cgit 1.2.3-korg