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