Review Request 124534: ksmserver: restore support for autostart scripts; migrate them from the KDE4 dir

David Faure faure at kde.org
Fri Jul 31 08:32:44 UTC 2015


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

(Updated July 31, 2015, 8:32 a.m.)


Review request for Plasma and Vishesh Handa.


Changes
-------

extract function; remove spaces within ()


Repository: plasma-workspace


Description (updated)
-------

Commit f913e251fe6 removed this, due to a porting bug: both klauncher
(XDG autostart using .desktop files) and ksmserver (kde-specific scripts)
were ported to look at the same directory (~/.config/autostart),
leading to double autostart. The right fix, however, is to use
a different directory for scripts, I called it ~/.config/autostart-scripts.

BUG: 338242
REVIEW: 124534


Diffs (updated)
-----

  ksmserver/server.h 2176aa1c9a6505773b61354dee6fd547fdf5841e 
  ksmserver/startup.cpp 531e88b4257901e890f16c23761d2c0aa538d524 

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


Testing
-------

Logged out and back in many times :)

My autostart folder got copied correctly, including symlinks


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150731/479e1712/attachment.html>


More information about the Plasma-devel mailing list