Review Request 119522: readlink fails whens the folder does not exist.
Rohan Garg
rohangarg at kubuntu.org
Mon Jul 28 17:48:54 UTC 2014
> On July 28, 2014, 11:14 p.m., Aleix Pol Gonzalez wrote:
> > So does it still work if it's a link, after this patch?
Yep, works when my ~/.config is a symlink.
- Rohan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119522/#review63351
-----------------------------------------------------------
On July 28, 2014, 10:47 p.m., Rohan Garg wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119522/
> -----------------------------------------------------------
>
> (Updated July 28, 2014, 10:47 p.m.)
>
>
> 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/05246faa/attachment-0001.html>
More information about the Plasma-devel
mailing list