D17816: Support for xattrs on kio copy/move

Cochise César noreply at phabricator.kde.org
Fri Mar 13 14:08:06 GMT 2020


cochise updated this revision to Diff 77559.
cochise added a comment.


  As @dfaure sugested, moved the attribute copy logic to a function on file_unix.cpp.
  
  As a side effect we lost hability to copy extened attributes of directories, (they are not copied, but created in CopyJob but now all tests pass.
  
  Minor changes:
  
  - Added tests for links, as the code works with them
  
  - Removed tests for copy/move to another partition, as this shpould fail. The destination don't support xattrs.

REPOSITORY
  R241 KIO

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D17816?vs=72144&id=77559

BRANCH
  arcpatch-D17816

REVISION DETAIL
  https://phabricator.kde.org/D17816

AFFECTED FILES
  autotests/jobtest.cpp
  autotests/jobtest.h
  src/ioslaves/file/ConfigureChecks.cmake
  src/ioslaves/file/file.h
  src/ioslaves/file/file_unix.cpp

To: cochise, dfaure, chinmoyr, bruns, #frameworks, tmarshall
Cc: scheirle, anthonyfieroni, tmarshall, arrowd, cfeck, bruns, phidrho, dhaumann, funkybomber, abika, pino, davidedmundson, ngraham, atha.kane, spoorun, nicolasfella, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200313/75adde9f/attachment.html>


More information about the Kde-frameworks-devel mailing list