D17632: Move/copy job: skip stat'ing sources if the destination dir isn't writable
Chinmoy Ranjan Pradhan
noreply at phabricator.kde.org
Mon Dec 17 15:38:45 GMT 2018
chinmoyr added inline comments.
INLINE COMMENTS
> copyjob.cpp:419
> + // TODO: this is done only for local files, but we could use the UDSEntry to do this portably
> + // ... assuming all kioslaves set permissions correctly
> + if (fileInfo.exists() && !fileInfo.isWritable()) {
For file permissions, 'file:' and 'trash:' use the result of QT_LSTAT, 'archive:' makes use of stat, 'sftp:' uses sftp_lstat, and smb sets UDS_ACCESS to 0500. These are the kioslaves(I found) which KDE ships and set UDS_ACCESS entry. I think using UDSEntry can be considered here.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D17632
To: dfaure, shubham, broulik, bruns
Cc: chinmoyr, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181217/5950cf63/attachment.html>
More information about the Kde-frameworks-devel
mailing list