Merge lp:~xnox/ubuntu-release-upgrader/lp1749199 into lp:ubuntu-release-upgrader

Proposed by Dimitri John Ledkov
Status: Merged
Merged at revision: 3138
Proposed branch: lp:~xnox/ubuntu-release-upgrader/lp1749199
Merge into: lp:ubuntu-release-upgrader
Diff against target: 57 lines (+14/-2)
4 files modified
DistUpgrade/DistUpgradeConfigParser.py (+2/-2)
data/DistUpgrade.cfg (+1/-0)
data/DistUpgrade.cfg.xenial (+1/-0)
debian/changelog (+10/-0)
To merge this branch: bzr merge lp:~xnox/ubuntu-release-upgrader/lp1749199
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+341342@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

This looks fine but the PurgeObsoletes changes should also be made in DistUpgrade.cfg.xenial particularly because the bug is about an upgrade from Xenial to Bionic.

review: Needs Fixing
3107. By Julian Andres Klode

Use HTTPS for changelogs.ubuntu.com (LP: #1744318)

3108. By Julian Andres Klode

releasing package ubuntu-release-upgrader version 1:18.04.10

3109. By Julian Andres Klode

Check that a sources.list entry for the from distribution exists
before attempting to upgrade (LP: #1725359)

3110. By Julian Andres Klode

releasing package ubuntu-release-upgrader version 1:18.04.11

3111. By Brian Murray

Merge Simon Quigley's migration from kdesudo to pkexec.

3112. By Brian Murray

change to unreleased until autopkgtests are sorted.

3113. By Brian Murray

Merge laney's change to unset a system's APT::Default-Release

3114. By Brian Murray

Increase the size of the buffer used when calculating the free space to
estimate for the initramfs.

3115. By Brian Murray

releasing package ubuntu-release-upgrader version 1:18.04.12

3116. By Launchpad Translations on behalf of ubuntu-core-dev

Launchpad automatic translations update.

3117. By Julian Andres Klode

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)

3118. By Julian Andres Klode

releasing package ubuntu-release-upgrader version 1:18.04.13

3120. By Dimitri John Ledkov

Also set PurgeObsoletes=yes in the .xenial config

3121. By Dimitri John Ledkov

Make dist-upgrade.py work when PWD != datadir, by fixing the logic for
finding and loading .release config files with datadir prepended
first.

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Updated the .xenial config.
Fixed the "discovery" of .distro config overrides.
Checked that the upgrades work from:
- bzr checkout
- dist-upgrader tarball

(which moves the .distro configs into the DistUpgrade folder / flat file structure, with default datadir set as '.' the previous logic hence simply worked by ignoring datadir and using current dir which was accidentally correct things to do)

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

This looks good to me now, thanks!

review: Approve
3122. By Dimitri John Ledkov

This line was intentionally left blank.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'DistUpgrade/DistUpgradeConfigParser.py'
2--- DistUpgrade/DistUpgradeConfigParser.py 2015-01-20 22:30:06 +0000
3+++ DistUpgrade/DistUpgradeConfigParser.py 2018-04-03 08:45:27 +0000
4@@ -40,9 +40,9 @@
5 ["lsb_release", "-c", "-s"], stdout=subprocess.PIPE,
6 universal_newlines=True).communicate()[0].strip()
7 self.datadir = datadir
8- if os.path.exists(name + "." + from_release):
9- name = name + "." + from_release
10 maincfg = os.path.join(datadir, name)
11+ if os.path.exists(maincfg + "." + from_release):
12+ maincfg += "." + from_release
13 # defaults are read first
14 self.config_files = []
15 if defaults_dir:
16
17=== modified file 'data/DistUpgrade.cfg'
18--- data/DistUpgrade.cfg 2018-02-05 11:26:33 +0000
19+++ data/DistUpgrade.cfg 2018-04-03 08:45:27 +0000
20@@ -23,6 +23,7 @@
21 KeepInstalledPkgs=xserver-xorg-video-all
22 KeepInstalledSection=translations
23 RemoveObsoletes=yes
24+PurgeObsoletes=yes
25 #ForcedObsoletes=ksplash-engine-moodin, powernowd, laptop-mode-tools
26 # hints for for stuff that should be done right after the cache calculated
27 # the dist-upgrade (not to be confused with the PostUpgrade hooks in the
28
29=== modified file 'data/DistUpgrade.cfg.xenial'
30--- data/DistUpgrade.cfg.xenial 2017-11-06 16:22:56 +0000
31+++ data/DistUpgrade.cfg.xenial 2018-04-03 08:45:27 +0000
32@@ -23,6 +23,7 @@
33 KeepInstalledPkgs=xserver-xorg-video-all
34 KeepInstalledSection=translations
35 RemoveObsoletes=yes
36+PurgeObsoletes=yes
37 #ForcedObsoletes=ksplash-engine-moodin, powernowd, laptop-mode-tools
38 # hints for for stuff that should be done right after the cache calculated
39 # the dist-upgrade (not to be confused with the PostUpgrade hooks in the
40
41=== modified file 'debian/changelog'
42--- debian/changelog 2018-03-29 13:30:23 +0000
43+++ debian/changelog 2018-04-03 08:45:27 +0000
44@@ -1,3 +1,13 @@
45+ubuntu-release-upgrader (1:18.04.14) UNRELEASED; urgency=medium
46+
47+ * Purge obsolete packages, if user agrees to remove obsolete
48+ packages. LP: #1749199
49+ * Make dist-upgrade.py work when PWD != datadir, by fixing the logic for
50+ finding and loading .release config files with datadir prepended
51+ first.
52+
53+ -- Dimitri John Ledkov <xnox@ubuntu.com> Fri, 30 Mar 2018 09:32:47 +0100
54+
55 ubuntu-release-upgrader (1:18.04.13) bionic; urgency=medium
56
57 * Do not consider Suggests important, so automatically installed packages

Subscribers

People subscribed via source and target branches