Review Request: Copy files instead of moving if parent dir is not writable

Lamarque Vieira Souza lamarque at kde.org
Tue Mar 27 15:22:47 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104417/
-----------------------------------------------------------

(Updated March 27, 2012, 2:22 p.m.)


Review request for KDE Runtime and Plasma.


Changes
-------

Changing patch to do not move files/folders if their parent folders are not writable. At least in Linux/Unix if the parent folder is not writable in means the user does not have permission to move the file (he/she may have permission to copy it though).


Description
-------

When adding an application resource to a private activity kactivitymanager tries to move the resource's .desktop file to the activity's private folder. The new .desktop file is created successfully but the source file is not deleted if the user does not have write permission on the file's directory. This patch detects such situation and uses copy instead of move to prevent "permission denied" messages for every resource being added.


This addresses bug 296808.
    http://bugs.kde.org/show_bug.cgi?id=296808


Diffs (updated)
-----

  service/jobs/nepomuk/Move.h 8a8afd1 
  service/jobs/nepomuk/Move.cpp 08a3cc2 

Diff: http://git.reviewboard.kde.org/r/104417/diff/


Testing
-------

Works on Meego devel image. The file is copied and no error message is shown.


Thanks,

Lamarque Vieira Souza

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120327/d9b84fa3/attachment.htm>


More information about the kde-core-devel mailing list