| Message ID | 20230711232819.2212134-3-p.lobacz@welotec.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/3] bitbake.conf: add acl and xattr distro native features support | expand |
On 12 Jul 2023, at 00:28, Piotr Łobacz via lists.openembedded.org <p.lobacz=welotec.com@lists.openembedded.org> wrote: > - # Preserve sparse files and hard links > - cmd = 'tar --exclude=./sysroot-only -cf - -C %s -p -S . | tar -xf - -C %s' % (dest, dvar) > + # Preserve sparse files, hard links, ACLs and extended attributes > + acl = '' > + xattr = '' > + # TODO: for the moment only ipk packages are supporting ACLs and extended attributes > + if bb.utils.contains('PACKAGE_CLASSES', 'package_ipk', True, False, d): > + acl = bb.utils.contains('DISTRO_FEATURES', 'acl', '--acls', '', d) > + xattr = bb.utils.contains('DISTRO_FEATURES', 'xattr', '--xattrs', '', d) > + cmd = f'tar {acl} {xattr} --numeric-owner --exclude=./sysroot-only -cf - -C {dest} -p -S . | tar {acl} {xattr} -xf - -C {dvar}' > subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) This is still changing the behaviour for rpm packages if ipkg is enabled, which sounds like a wonderful way to introduce subtle bugs that are hard to find. Is there a problem with always passing —acls —xatttrs to tar at this step? Ross
Hmmmm actually no, we can always pass these params to tar. Wysyłane z aplikacji Outlook dla systemu iOS<https://aka.ms/o0ukef>
diff --git a/meta/classes-global/package.bbclass b/meta/classes-global/package.bbclass index e8055a9cdc..7229e4f2b5 100644 --- a/meta/classes-global/package.bbclass +++ b/meta/classes-global/package.bbclass @@ -342,8 +342,14 @@ python perform_packagecopy () { # Start by package population by taking a copy of the installed # files to operate on - # Preserve sparse files and hard links - cmd = 'tar --exclude=./sysroot-only -cf - -C %s -p -S . | tar -xf - -C %s' % (dest, dvar) + # Preserve sparse files, hard links, ACLs and extended attributes + acl = '' + xattr = '' + # TODO: for the moment only ipk packages are supporting ACLs and extended attributes + if bb.utils.contains('PACKAGE_CLASSES', 'package_ipk', True, False, d): + acl = bb.utils.contains('DISTRO_FEATURES', 'acl', '--acls', '', d) + xattr = bb.utils.contains('DISTRO_FEATURES', 'xattr', '--xattrs', '', d) + cmd = f'tar {acl} {xattr} --numeric-owner --exclude=./sysroot-only -cf - -C {dest} -p -S . | tar {acl} {xattr} -xf - -C {dvar}' subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT) # replace RPATHs for the nativesdk binaries, to make them relocatable
Extend `tar` command, with additional parameters, depending on choosen package class and target distro features, in order to support ACLs and xattr. Currently only `package_ipk` supports fully ACLs and xattr. Signed-off-by: Piotr Łobacz <p.lobacz@welotec.com> --- meta/classes-global/package.bbclass | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)