DevHeads.net

Review Request 121058: Don't put scripts from ~/.config/autostart in autostart kcm list.

Review request for kdelibs, Plasma and Luca Beltrame.

Repository: plasma-desktop

Description
KLauncher doesn't start these, so we shouldn't list them.

Diffs
kcms/autostart/autostart.cpp 8fe81ad7fc752936cd8ba1c799ad06bafd2c4006

Diff: <a href="https://git.reviewboard.kde.org/r/121058/diff/" title="https://git.reviewboard.kde.org/r/121058/diff/">https://git.reviewboard.kde.org/r/121058/diff/</a>

Testing
an executable script (ssh-add.sh) in ~/.config/autostart/ doesn't appear in the kcm here anymore.

Thanks,

Jeremy Whiting

Comments

Re: Review Request 121058: Don't put scripts from ~/.config/auto

By =?utf-8?Q?Thoma... at 11/07/2014 - 17:26

kcms/autostart/autostart.cpp
<https://git.reviewboard.kde.org/r/121058/#comment48973>

maybe cache that in a bool in the outer loop

- Thomas L├╝bking

On Nov. 7, 2014, 10:01 nachm., Jeremy Whiting wrote: