On 12/05/2025 18:11, Paul Eggert wrote:
On 2025-05-12 05:43, Ondrej Valousek wrote:
I think they intentionally removed the system.nfs4* in Fedora-42 because commands like
"cp -a" complained when copying files.
Yes, the removal in 2019 was due to a suggestion[1] by Pádraig, which
Kamil Dudka was a bit dubious about[2] but went ahead and did anyway[3].
I'll cc Kamil to see whether he has further comments now.
What we can (and should) do is a check whether the ACLs detected are primitive
(i.e. no ACLs at all) and in this case, just skip the ACL copy.
The Gnulib patch[4] I recently installed does something like that,
except that for efficiency in the non-NFS case it checks whether the
ACLs are primitive only if the ACL copy (via attr_copy_file or
attr_copy_fd) fails with EOPNOTSUP.
With [4] in place, I guess Pádraig's point is that Bruno's older Gnulib
patch[5] is no longer needed. Although I'm not sure about that, I
confess that I'm a bit lost.
What a mess, huh?
Right.
Given the responses to this thread, I think the current situation is best.
I.e. hardcoded handling of NFSv4 xattrs in gnulib, and not including them in
xattr.conf.
I.e. these xattrs require nuanced handling (which gnulib now does),
while entries in xattr.conf are more appropriate for simpler xattrs
that one can simply copy.
cheers,
Pádraig.
