D9288: support for defining autostart apps in lnf packages

2017-12-13 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R119:49ae81f280bd: support for defining autostart apps in lnf packages (authored by mart). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D9288?vs=23869=23870#toc REPOSITORY R119 Plasma Desktop

D9288: support for defining autostart apps in lnf packages

2017-12-13 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH phab/lnfautostartwq! REVISION DETAIL https://phabricator.kde.org/D9288 To: mart, #plasma, davidedmundson Cc: ngraham, mvourlakos, apol, plasma-devel, ZrenBot,

D9288: support for defining autostart apps in lnf packages

2017-12-13 Thread Marco Martin
mart updated this revision to Diff 23869. mart added a comment. - only mess with autostart if we apply the layout REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9288?vs=23805=23869 BRANCH phab/lnfautostartwq! REVISION DETAIL

D9288: support for defining autostart apps in lnf packages

2017-12-12 Thread Michail Vourlakos
mvourlakos added a comment. In https://phabricator.kde.org/D9288#178638, @mart wrote: > we could add a new named file in the lnf package for the latte layout: in what format is it exported? > so lattedock upon startup may go read that file (is already possible for any app to access

D9288: support for defining autostart apps in lnf packages

2017-12-12 Thread Marco Martin
mart updated this revision to Diff 23805. mart added a comment. - e Qtring::section REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9288?vs=23764=23805 BRANCH phab/lnfautostartwq! REVISION DETAIL https://phabricator.kde.org/D9288 AFFECTED

D9288: support for defining autostart apps in lnf packages

2017-12-12 Thread Marco Martin
mart marked 2 inline comments as done. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D9288 To: mart, #plasma Cc: ngraham, mvourlakos, apol, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart

D9288: support for defining autostart apps in lnf packages

2017-12-12 Thread Marco Martin
mart added inline comments. INLINE COMMENTS > apol wrote in kcm.cpp:387 > Also I'm not sure if it works, but something along the lines of `systemctl > --user stop servicename` should work. Or at least it's a thing. it seems to look for a systemd service file? REPOSITORY R119 Plasma Desktop

D9288: support for defining autostart apps in lnf packages

2017-12-12 Thread Marco Martin
mart added a comment. In https://phabricator.kde.org/D9288#178442, @mvourlakos wrote: > in the future would it be possible for these apps to include also their layout in the Look N' Feel package? > For example Latte supports import/export of its layouts so it is possible to

D9288: support for defining autostart apps in lnf packages

2017-12-11 Thread Michail Vourlakos
mvourlakos added a comment. @mart I suppose this mechanism is the first step in order to load external apps (such as lattedock) through the Look N' Feel package. Fantastic news of course! :) in the future would it be possible for these apps to include also their layout in the Look N'

D9288: support for defining autostart apps in lnf packages

2017-12-11 Thread Aleix Pol Gonzalez
apol added inline comments. INLINE COMMENTS > kcm.cpp:387 > +//FIXME: quite ugly way to stop things, and what about non > KDE things? > +QProcess::startDetached(QStringLiteral("kquitapp5"), >

D9288: support for defining autostart apps in lnf packages

2017-12-11 Thread Marco Martin
mart created this revision. mart added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY meant to support layouts that want external apps such as lattedock or conky, it adds services (in the defaults