Review Request 119522: readlink fails whens the folder does not exist.
Rohan Garg
rohangarg at kubuntu.org
Mon Jul 28 17:17:25 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119522/
-----------------------------------------------------------
Review request for Plasma and Àlex Fiestas.
Repository: plasma-workspace
Description
-------
https://projects.kde.org/projects/kde/kde-workspace/repository/revisions/f01badb5aa3f76b27955f4fee2cb7281dd490353 introduced a regression in startkde whereby if ~/.config does not exist readlink will not output the right path.
This leads to startkde trying to write to /startupconfigkeys or /kdeglobals which obviously won't work.
Since the original commit mentions that relative paths weren't working, I've taken a alternative approach of using ${HOME} instead.
Diffs
-----
startkde/startkde.cmake 281bd43
Diff: https://git.reviewboard.kde.org/r/119522/diff/
Testing
-------
Kubuntu Plasma 5 ISO's did not have the right cursor theme before this patch, after editing startkde by hand, the right files were being correctly written, and the cursor theme worked as expected.
Thanks,
Rohan Garg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140728/0adbef56/attachment.html>
More information about the Plasma-devel
mailing list