D26116: Reduce timeout when trying to connect to an app dbus interface

2019-12-20 Thread Méven Car
meven added a comment. In D26116#580539 , @davidedmundson wrote: > > Encountered this issue while testing krunner. > > Can you expand on this, so I can see what problem we're actually trying to solve. I had a "org.kde.krunner" servic

D26116: Reduce timeout when trying to connect to an app dbus interface

2019-12-20 Thread David Edmundson
davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. > I would say any application that does not respond in 5 seconds is buggy, or at least in a bad state. That's not necessarily true. We're asking an app to quit

D26116: Reduce timeout when trying to connect to an app dbus interface

2019-12-20 Thread Méven Car
meven added a comment. In D26116#580480 , @broulik wrote: > I'd say 5 seconds is optimistic. Granted. 10 seconds maybe. > Does Kate's session selection dialog still block startup? I am not aware of this. REPOSITORY R271 KDBu

D26116: Reduce timeout when trying to connect to an app dbus interface

2019-12-20 Thread Kai Uwe Broulik
broulik added a comment. I'd say 5 seconds is optimistic. Does Kate's session selection dialog still block startup? REPOSITORY R271 KDBusAddons REVISION DETAIL https://phabricator.kde.org/D26116 To: meven, #frameworks, davidedmundson Cc: broulik, kde-frameworks-devel, LeGast00n, GB_2,

D26116: Reduce timeout when trying to connect to an app dbus interface

2019-12-19 Thread Méven Car
meven updated this revision to Diff 71867. meven added a comment. Include kde dbus daemon case REPOSITORY R271 KDBusAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26116?vs=71866&id=71867 BRANCH master REVISION DETAIL https://phabricator.kde.org/D26116 AFFECTED FILES

D26116: Reduce timeout when trying to connect to an app dbus interface

2019-12-19 Thread Méven Car
meven created this revision. meven added reviewers: Frameworks, davidedmundson. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY Currently we have a 5 minutes timeout duration. I would say any applica