Commit dd9ba961 authored by Aleksei Nikiforov's avatar Aleksei Nikiforov Committed by Aleix Pol Gonzalez
Browse files

discover: check package status before trying to install/update it

Summary:
With latest changes to packagekit, it may now return more packages from 'pkcon get-updates' command with new statuses: "removed" or "obsoleted". This change is needed to filter such packages since Discover is using list of packages similar to the output of mentioned command but ignores package statuses. Otherwise, Discover would try to install all packages, including removed and obsoleted, and fail while trying to resolve such transaction due to package conflicts, if there are such packages present.

packagekit changes:
https://github.com/hughsie/PackageKit/pull/341

Reviewers: jriddell, apol

Reviewed By: apol

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D24653
parent 1105ee0c
......@@ -583,6 +583,11 @@ void PackageKitBackend::addPackageToUpdate(PackageKit::Transaction::Info info, c
return;
}
if (info == PackageKit::Transaction::InfoRemoving || info == PackageKit::Transaction::InfoObsoleting) {
// Don't try updating packages which need to be removed
return;
}
if (info == PackageKit::Transaction::InfoSecurity)
m_hasSecurityUpdates = true;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment