Merge lp:~juliank/ubuntu-release-upgrader/suggests-not-important into lp:ubuntu-release-upgrader

Proposed by Julian Andres Klode
Status: Merged
Merged at revision: 3117
Proposed branch: lp:~juliank/ubuntu-release-upgrader/suggests-not-important
Merge into: lp:ubuntu-release-upgrader
Diff against target: 29 lines (+11/-0)
2 files modified
DistUpgrade/DistUpgradeCache.py (+3/-0)
debian/changelog (+8/-0)
To merge this branch: bzr merge lp:~juliank/ubuntu-release-upgrader/suggests-not-important
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+341655@code.launchpad.net

Description of the change

Do not consider Suggests important, so automatically installed packages
that are not recommended or dependend on anymore are removed across a
dist upgrade (LP: #1725861).

Not tested whether it actually works, but it seems reasonable to assume so. Not sure how to test it.

To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

This seems fine, but I'm testing it on a system that's been upgraded over multiple releases which when manually using the setting will autoremove a number of packages.

Revision history for this message
Brian Murray (brian-murray) wrote :

The "Remove (was auto installed)" list includes packages I'd seen when using apt autoremove with the setting in effect before the upgrade so it seems like its doing the right thing.

Another involved way to test would be to install zesty, upgrade to artful and then ensure "ubuntu-wallpapers-zesty" was removed.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'DistUpgrade/DistUpgradeCache.py'
2--- DistUpgrade/DistUpgradeCache.py 2016-12-22 00:02:04 +0000
3+++ DistUpgrade/DistUpgradeCache.py 2018-03-19 14:27:00 +0000
4@@ -145,6 +145,9 @@
5 not "RELEASE_UPGRADE_NO_RECOMMENDS" in os.environ):
6 apt_pkg.config.set("APT::Install-Recommends", "true")
7
8+
9+ apt_pkg.config.set("APT::AutoRemove::SuggestsImportant", "false")
10+
11 def _apply_dselect_upgrade(self):
12 """ honor the dselect install state """
13 for pkg in self:
14
15=== modified file 'debian/changelog'
16--- debian/changelog 2018-03-16 17:56:35 +0000
17+++ debian/changelog 2018-03-19 14:27:00 +0000
18@@ -1,3 +1,11 @@
19+ubuntu-release-upgrader (1:18.04.12) UNRELEASED; urgency=medium
20+
21+ * Do not consider Suggests important, so automatically installed packages
22+ that are not recommended or dependend on anymore are removed across a
23+ dist upgrade (LP: #1725861)
24+
25+ -- Julian Andres Klode <juliank@ubuntu.com> Mon, 19 Mar 2018 15:19:12 +0100
26+
27 ubuntu-release-upgrader (1:18.04.11) bionic; urgency=medium
28
29 * Check that a sources.list entry for the from distribution exists

Subscribers

People subscribed via source and target branches