diff options
author | Joshua Lock <joshua.g.lock@intel.com> | 2016-09-01 17:26:41 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-09-03 09:58:42 +0100 |
commit | 92f9308e2cadcd47ca66e6cc08aa7b1df261b0cc (patch) | |
tree | aa42cee770569544c6efee7c33754dd1c629ccbf /meta/lib/oe | |
parent | 97677c11cf186a17439c742788a2914578d9dc99 (diff) | |
download | poky-92f9308e2cadcd47ca66e6cc08aa7b1df261b0cc.tar.gz |
oe.path: preserve xattr in copytree() and copyhardlinktree()
Pass appropriate options to tar invocations in copytree() and
copyhardlinktree() to ensure that any extended attributes on the files
are preserved during the copy.
We have to drop the use cpio in "Copy-pass" mode in copyhardlinktree()
because cpio doesn't support extended attributes on files. Instead we
revert back to using cp with different patterns depending on whether
or not the directory contains dot files.
(From OE-Core rev: e591d69103a40ec4f76d1132a6039d9cb1555103)
Signed-off-by: Joshua Lock <joshua.g.lock@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oe')
-rw-r--r-- | meta/lib/oe/path.py | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/meta/lib/oe/path.py b/meta/lib/oe/path.py index 3c07df33d4..631c3b430c 100644 --- a/meta/lib/oe/path.py +++ b/meta/lib/oe/path.py | |||
@@ -65,7 +65,7 @@ def copytree(src, dst): | |||
65 | # This way we also preserve hardlinks between files in the tree. | 65 | # This way we also preserve hardlinks between files in the tree. |
66 | 66 | ||
67 | bb.utils.mkdirhier(dst) | 67 | bb.utils.mkdirhier(dst) |
68 | cmd = 'tar -cf - -C %s -p . | tar -xf - -C %s' % (src, dst) | 68 | cmd = "tar --xattrs --xattrs-include='*' -cf - -C %s -p . | tar --xattrs --xattrs-include='*' -xf - -C %s" % (src, dst) |
69 | subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) | 69 | subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) |
70 | 70 | ||
71 | def copyhardlinktree(src, dst): | 71 | def copyhardlinktree(src, dst): |
@@ -77,9 +77,14 @@ def copyhardlinktree(src, dst): | |||
77 | if (os.stat(src).st_dev == os.stat(dst).st_dev): | 77 | if (os.stat(src).st_dev == os.stat(dst).st_dev): |
78 | # Need to copy directories only with tar first since cp will error if two | 78 | # Need to copy directories only with tar first since cp will error if two |
79 | # writers try and create a directory at the same time | 79 | # writers try and create a directory at the same time |
80 | cmd = 'cd %s; find . -type d -print | tar -cf - -C %s -p --no-recursion --files-from - | tar -xf - -C %s' % (src, src, dst) | 80 | 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) |
81 | subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) | 81 | subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) |
82 | cmd = 'cd %s; find . -print0 | cpio --null -pdlu %s' % (src, dst) | 82 | if os.path.isdir(src): |
83 | import glob | ||
84 | if len(glob.glob('%s/.??*' % src)) > 0: | ||
85 | src = src + '/.??* ' | ||
86 | src = src + '/*' | ||
87 | cmd = 'cp -afl --preserve=xattr %s %s' % (src, dst) | ||
83 | subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) | 88 | subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) |
84 | else: | 89 | else: |
85 | copytree(src, dst) | 90 | copytree(src, dst) |