D21666: Do not match daemonized dolphin instances
Elvis Angelaccio
noreply at phabricator.kde.org
Sat Jun 8 12:01:52 BST 2019
elvisangelaccio created this revision.
elvisangelaccio added reviewers: broulik, ngraham, feverfew.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
elvisangelaccio requested review of this revision.
REVISION SUMMARY
`dolphin --daemon` does not have the `/dolphin/Dolphin_1` dbus path,
because it doesn't have any DolphinMainWindow.
Checking if the QDBusInterface is valid is not enough, we also need to look for dbus errors.
TEST PLAN
1. Make sure there is a `dolphin --deamon` process around.
2. Start another dolphin process.
REPOSITORY
R318 Dolphin
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D21666
AFFECTED FILES
src/global.cpp
To: elvisangelaccio, broulik, ngraham, feverfew
Cc: kfm-devel, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190608/d63e98a9/attachment.htm>
More information about the kfm-devel
mailing list