D17816: Support for xattrs on kio copy/move

Stefan BrĂ¼ns noreply at phabricator.kde.org
Tue Sep 8 18:15:05 BST 2020


bruns added inline comments.

INLINE COMMENTS

> jobtest.cpp:467
> +    path.removeLast();
> +    QString writeTest = path.join("/") + "/fsXattrTestFile";
> +    createTestFile(writeTest);

const QString

And if you just call this with the target directory (e.g. ` homeTmpDir()`), you can skip the split/join dance.

> jobtest.cpp:780
> +    checkXattrFsSupport(filePath);
> +    checkXattrFsSupport(dest);
> +    if (!m_SkipXattr) {

This will set `m_SkipXattr` unconditionally even if the source FS does not support Xattrs.

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/20200908/84d9f41d/attachment-0001.htm>


More information about the Kde-frameworks-devel mailing list