Skip to content

Fix displaying notification of Additional Software installation failure

intrigeri requested to merge 20170-fix-asp-install-failure-notification into testing

a73ebb0f was not enough to fix the regression introduced by d723561e: we did not support all the non-zero success exit codes, so notify() would return None, then asp-install will try and fail to close that notification (by lack of a valid ID), so finally the following notification about failure to install will (most of the time?) fail to override the previous ("Installing") notification.

refs #20170 (closed)

Closes #20170 (closed)

Merge request reports