D28836: Dolphin Service Installer: Run uninstall scripts, rename methods

Alexander Lohnau noreply at phabricator.kde.org
Tue Apr 14 18:53:15 BST 2020


alex created this revision.
alex added reviewers: Dolphin, ngraham, nicolasfella, elvisangelaccio.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
alex requested review of this revision.

REVISION SUMMARY
  Now scripts starting with `uninstall` are being executed.
  Addtionally the `runInstallerScriptOnce` method has been renamed, because it is used to run the install/uninstall scripts.
  Same for `runInstallerScript`.
  Depends on D28801 <https://phabricator.kde.org/D28801>

TEST PLAN
  Compile, run tests. Try to uninstall a dolphin service which has a uninstall.sh script.

REPOSITORY
  R318 Dolphin

BRANCH
  allow_uninstall

REVISION DETAIL
  https://phabricator.kde.org/D28836

AFFECTED FILES
  src/settings/services/servicemenuinstaller/servicemenuinstaller.cpp

To: alex, #dolphin, ngraham, nicolasfella, elvisangelaccio
Cc: kfm-devel, azyx, nikolaik, pberestov, iasensio, fprice, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200414/01335e13/attachment.htm>


More information about the kfm-devel mailing list