On 2025-05-13 06:26, Bruno Haible via Gnulib discussion list wrote:
Paul Eggert wrote:
- || strcmp (name, XATTR_NAME_NFSV4_ACL) == 0
- || attr_copy_action (name, ctx) == ATTR_ACTION_PERMISSIONS;
+ || strcmp (name, XATTR_NAME_NFSV4_ACL) == 0;
}
#endif /* USE_XATTR */
xattr.conf may also list_additional_ attributes in the future, which we
cannot imagine yet. Removing the call to attr_copy_action would mean to
make our code less future-proof.
True, so perhaps we should leave the Gnulib code alone.
Still, it bugs me that cp, mv etc. essentially ignoe Fedora's
/etc/xattr.conf in practice. And they ignore Debian's /etc/xattr.conf
too, except for its line "system.nfs4acl permissions" (which lacks an
underscore between "nfs4" and "acl"). I don't know why Debian xattr.conf
has that line in addition to "system.nfs4_acl permissions" (with an
underscore), and I don't know why Gnulib hardwires the latter but not
the former line.
Ideally, this should be consistent. If there's a real need for the
underscore-free "system.nfs4acl permissions" line in Debian, then that
line should also be hardwired in Gnulib qcopy-acl; if not, that line
should be removed from Debian xattr.conf. Either way, Fedora's
xattr.conf should be changed to match Debian's, as the only reason
Fedora's omits NFS ACLs was because Coreutils/Gnulib was too picky about
copying those ACLs but that bug has now been fixed.
