hchain added inline comments. INLINE COMMENTS
> davidedmundson wrote in kde-systemd-start-condition.cpp:17 > Should it be showHelp(255)? > > Not that we should end up here in the generated case. I intentionally return 0 here to mimic the behavior of KAutostart which returns true if the condition is malformed or empty > davidedmundson wrote in kde-systemd-start-condition.cpp:20 > A group name can have a space in it. > > For example: > X-KDE-autostart-condition=baloofilerc:Basic Settings:Indexing-Enabled:true > > I assume this will be ok as long as we make sure that we make the generator > put things in quotes. But can you double check. so I could silently join the arguments with spaces ? REPOSITORY R309 KService REVISION DETAIL https://phabricator.kde.org/D28421 To: hchain, davidedmundson, broulik, dfaure, kossebau Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns