diff options
author | Katja Marttila <katja.marttila@qt.io> | 2020-09-23 19:37:31 +0300 |
---|---|---|
committer | Katja Marttila <katja.marttila@qt.io> | 2020-09-25 10:32:30 +0300 |
commit | 02f8711dc250b4c4f07ea7b43cbb0904496b1cb3 (patch) | |
tree | 91357567cd4dfe0c0df14e2b21f5ef871fcce14e /src/libs/installer/packagemanagercore_p.cpp | |
parent | 557feae54796a861f6076d30df17958e14debb4c (diff) |
CLI: Give more meaningfull return value for installs
Instead of returning just true or false when running installer or
maintenancetool, utilize the PackagemanagerCore status message.
Added also a new status enum, EssentialUpdated, which is returned when
calling command 'update' and only essential components are updated.
Also fixed a bug when components could be installed even when there
were an essential update available.
Task-number: QTIFW-1969
Change-Id: I43826301656573b34e1338b49566d199bdcd7468
Reviewed-by: Arttu Tarkiainen <arttu.tarkiainen@qt.io>
Diffstat (limited to 'src/libs/installer/packagemanagercore_p.cpp')
-rw-r--r-- | src/libs/installer/packagemanagercore_p.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/libs/installer/packagemanagercore_p.cpp b/src/libs/installer/packagemanagercore_p.cpp index da6ce66b9..52e732e49 100644 --- a/src/libs/installer/packagemanagercore_p.cpp +++ b/src/libs/installer/packagemanagercore_p.cpp @@ -1820,7 +1820,10 @@ bool PackageManagerCorePrivate::runPackageUpdater() if (adminRightsGained) m_core->dropAdminRights(); - setStatus(PackageManagerCore::Success); + if (m_foundEssentialUpdate) + setStatus(PackageManagerCore::EssentialUpdated); + else + setStatus(PackageManagerCore::Success); emit installationFinished(); } catch (const Error &err) { if (m_core->status() != PackageManagerCore::Canceled) { |