alex created this revision. alex added reviewers: KNewStuff, ngraham, leinir. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. alex requested review of this revision.
REVISION SUMMARY If the install script failed/was aborted the entry gets marked as installed. Now the exit code is checked and a error message is displayed. TEST PLAN Make sure to have D29119 <https://phabricator.kde.org/D29119>. Install the deb package of the "Jetbrains" dolphin addon. When the authorization popup comes press escape. The dolphin installer shows an error popup and the entry is not marked as installed. If you are not on a debian based system it should throw an error anyway ;-). REPOSITORY R304 KNewStuff BRANCH handle_install_script_error (branched from master) REVISION DETAIL https://phabricator.kde.org/D29451 AFFECTED FILES src/core/installation.cpp To: alex, #knewstuff, ngraham, leinir Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns