https://bugs.kde.org/show_bug.cgi?id=420312
Alex <alexander.loh...@gmx.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit|https://cgit.kde.org/knewst |https://commits.kde.org/kne |uff.git/commit/?id=4c6b3afc |wstuff/a2ecdd333c492d3e4be5 |d80a7b03e0381e2a5c8bb693a8f |c21301374073784f1b60 |916fd | --- Comment #13 from Alex <alexander.loh...@gmx.de> --- Git commit a2ecdd333c492d3e4be5c21301374073784f1b60 by Alexander Lohnau. Committed on 11/05/2020 at 18:39. Pushed by alex into branch 'master'. Do not mark entry as uninstalled if uninstallation script failed Summary: As described in the bug report the uninstallation failed, but the service was marked as removed. Now the service gets only marked as uninstalled if the script runs without an error. If there is an error the user gets a popup. Depends on D29101 Test Plan: Set the exit code to 1 and try to install a dolphin plugin. You should get an error message. Without modifying the exit code you should be able to install services. The manual deletion can be tested by removing the installed service file. for example: rm ~/.local/share/kservices5/ServiceMenus/iso_mounter_unmounter.desktop Then the uninstaller will crash: "Failed to remove .desktop file ... No such file or directory" Then you delete the installed file manually: rm ~/.local/share/servicemenu-download/iso_mounter_unmounter.desktop And now you can click the uninstall button and it gets removed from the list of installed services. Reviewers: #knewstuff, meven, ngraham, leinir Reviewed By: #knewstuff, ngraham, leinir Subscribers: leinir, kde-frameworks-devel Tags: #frameworks Differential Revision: https://phabricator.kde.org/D29123 M +1 -3 src/core/engine.cpp M +41 -10 src/core/installation.cpp M +3 -3 src/core/installation.h https://commits.kde.org/knewstuff/a2ecdd333c492d3e4be5c21301374073784f1b60 -- You are receiving this mail because: You are watching all bug changes.