Review Request 129729: [OS X] : don't risk deleting /Applications (!)

René J.V. Bertin rjvbertin at gmail.com
Fri Dec 30 19:54:50 UTC 2016


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

Review request for KDE Software on Mac OS X and KDE Frameworks.


Repository: kservice


Description
-------

I noticed that kbuildsyscoca.cpp may delete `QStandardPaths::writableLocation(QStandardPaths::ApplicationsLocation)`, according to the comments if that location is empty, but in practice no check is made whether that's the case.

And that makes the tool very dangerous on Mac, where the native ApplicationsLocation is something very different.


Diffs
-----

  src/sycoca/kbuildsycoca.cpp 5e63907 

Diff: https://git.reviewboard.kde.org/r/129729/diff/


Testing
-------

Works as intended (though I haven't tried to get it to fail, for some curious reason :))


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161230/306df92d/attachment.html>


More information about the Kde-frameworks-devel mailing list