--===============2277512140927264518== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/106911/ ----------------------------------------------------------- (Updated Sept. 24, 2016, 10:04 a.m.) Status ------ This change has been discarded. Review request for Plasma and Luboš Luňák. Bugs: 286658 http://bugs.kde.org/show_bug.cgi?id=286658 Repository: kde-workspace Description ------- It does not make sense to feed non-executable files to krun, since that folder is named as "Autostart", which expects executable files. Those .desktop files might be exception to the above rule, but I don't know whether the current situation of supporting .desktop under ~/.kde4/Autostart is an intentional or incidental feature. Diffs ----- ksmserver/startup.cpp d99ce4c Diff: https://git.reviewboard.kde.org/r/106911/diff/ Testing ------- Thanks, Jekyll Wu --===============2277512140927264518== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 8bit
This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/106911/

This change has been discarded.

Review request for Plasma and Luboš Luňák.
By Jekyll Wu.

Updated Sept. 24, 2016, 10:04 a.m.

Bugs: 286658
Repository: kde-workspace

Description 

It does not make sense to feed non-executable files to krun, since that folder is named as "Autostart", which expects executable files.

Those .desktop files might be exception to the above rule, but I don't know whether the current situation of supporting .desktop under ~/.kde4/Autostart is an intentional or incidental feature.

Diffs

  • ksmserver/startup.cpp (d99ce4c)

View Diff
--===============2277512140927264518==--