D17816: Support for xattrs on kio copy/move
David Faure
noreply at phabricator.kde.org
Fri Oct 30 07:37:39 GMT 2020
dfaure added inline comments.
INLINE COMMENTS
> arrowd wrote in ConfigureChecks.cmake:12
> > 1. enable the ACL stuff on systems with extattr, see the little bit of code in kpropertiesdialog.cpp
>
> By that you mean that I should edit the CMake module to define `HAVE_POSIX_ACL` when extattr headers are found?
>
> Or should I change checks in kpropertiesdialog.cpp from `HAVE_POSIX_ACL` to `HAVE_*ATTR_H`?
I'm talking about the implementation of fileSystemSupportsACL in kpropertiesdialog.cpp, which uses getxattr.
But now that I take another look at it, I see that it's already implemented on FreeBSD, without the use of extattr, it uses statfs instead.
So I think I was wrong, this part is fine.
I guess the only thing that's a bit weird is that kio_file's attr code (unrelated to ACLs) relies on FindACL linking to the right lib (on Linux). It would be cleaner if the attr stuff was separate. But no big deal I guess.
REVISION DETAIL
https://phabricator.kde.org/D17816
To: arrowd, dfaure, chinmoyr, bruns, #frameworks, tmarshall, usta, cochise
Cc: usta, scheirle, tmarshall, arrowd, cfeck, bruns, phidrho, dhaumann, funkybomber, abika, pino, davidedmundson, ngraham, atha.kane, spoorun, nicolasfella, kde-frameworks-devel, LeGast00n, cblack, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20201030/1aacfab7/attachment-0001.htm>
More information about the Kde-frameworks-devel
mailing list