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