D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

This revision was automatically updated to reflect the committed changes. Closed by commit R120:6cd25ded9934: Add a new standalone executable to replace X-KDE-Autostart-Condition (authored by hchain). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE

D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

davidedmundson added a comment. See https://github.com/systemd/systemd/pull/15226 REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28424 To: hchain, davidedmundson, broulik, dfaure, ngraham, meven Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack,

D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

meven added a comment. In D28424#637869 , @davidedmundson wrote: > looks good to me, but lets delay merging till the relevant other parts are ready. Seems ready to land ;) REPOSITORY R120 Plasma Workspace REVISION DETAIL

D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

davidedmundson added a comment. looks good to me, but lets delay merging till the relevant other parts are ready. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D28424 To: hchain, davidedmundson, broulik, dfaure, ngraham, meven Cc: plasma-devel, Orage,

D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

davidedmundson added inline comments. INLINE COMMENTS > broulik wrote in CMakeLists.txt:43 > Do we rather want it in libexec since it's not meant to be run by the user? There's a bit of a challenge there. We need this executable to be found by the xdg generator. libexec seems to be all over

D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

broulik added inline comments. INLINE COMMENTS > CMakeLists.txt:43 > +if (CMAKE_SYSTEM_NAME STREQUAL "Linux") > +install(TARGETS kde-systemd-start-condition DESTINATION > ${KDE_INSTALL_BINDIR}) > +endif () Do we rather want it in libexec since it's not meant to be run by the user?

D28424: Add a new standalone executable to replace X-KDE-Autostart-Condition

hchain created this revision. hchain added reviewers: davidedmundson, broulik, dfaure, ngraham, meven. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. hchain requested review of this revision. REVISION SUMMARY For systemd autostart generator will add a