Merge ~jibel/ubiquity/+git/ubiquity:lp1845593_fix_removed_section_attribute into ubiquity:master

Proposed by Jean-Baptiste Lallement
Status: Merged
Merged at revision: 28e140b0ffaabf3dc8bf9ade99462b472fa6efd8
Proposed branch: ~jibel/ubiquity/+git/ubiquity:lp1845593_fix_removed_section_attribute
Merge into: ubiquity:master
Diff against target: 42 lines (+11/-2)
3 files modified
debian/changelog (+7/-0)
scripts/install.py (+2/-1)
scripts/plugininstall.py (+2/-1)
Reviewer Review Type Date Requested Status
Iain Lane Approve
Julian Andres Klode (community) Approve
Review via email: mp+373302@code.launchpad.net

Commit message

Fix crash with free software only installation

scripts/install.py: Replaced removed property Package.section by Version.section (LP: #1845593)

To post a comment you must log in.
Revision history for this message
Julian Andres Klode (juliank) wrote :

Are we sure a candidate exists? I guess so, since there should not be any negative pins during installation, so approval from the python-apt side :)

review: Approve
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

Patch tested on a fresh installation of Eoan Desktop 20190926.1

Revision history for this message
Iain Lane (laney) wrote :

thanks a lot :>

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index a29c7d9..39b8442 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
1ubiquity (19.10.13) eoan; urgency=medium
2
3 * scripts/install.py: Replaced removed property Package.section by
4 Version.section (LP: #1845593)
5
6 -- Jean-Baptiste Lallement <jean-baptiste.lallement@ubuntu.com> Fri, 27 Sep 2019 09:44:57 +0200
7
1ubiquity (19.10.12) eoan; urgency=medium8ubiquity (19.10.12) eoan; urgency=medium
29
3 * ubiquity-dm: Still start gsd-xsettings for non-gnome-shell desktops.10 * ubiquity-dm: Still start gsd-xsettings for non-gnome-shell desktops.
diff --git a/scripts/install.py b/scripts/install.py
index 6848ed8..85e9922 100755
--- a/scripts/install.py
+++ b/scripts/install.py
@@ -265,7 +265,8 @@ class Install(install_misc.InstallBase):
265 if not use_restricted:265 if not use_restricted:
266 for pkg in cache.keys():266 for pkg in cache.keys():
267 if (cache[pkg].is_installed and267 if (cache[pkg].is_installed and
268 cache[pkg].section.startswith('restricted/')):268 cache[pkg].candidate.section.startswith(
269 'restricted/')):
269 difference.add(pkg)270 difference.add(pkg)
270271
271 # Keep packages we explicitly installed.272 # Keep packages we explicitly installed.
diff --git a/scripts/plugininstall.py b/scripts/plugininstall.py
index 93f852f..8292444 100755
--- a/scripts/plugininstall.py
+++ b/scripts/plugininstall.py
@@ -1414,7 +1414,8 @@ class Install(install_misc.InstallBase):
1414 cache = self.restricted_cache1414 cache = self.restricted_cache
1415 for pkg in cache.keys():1415 for pkg in cache.keys():
1416 if (cache[pkg].is_installed and1416 if (cache[pkg].is_installed and
1417 cache[pkg].section.startswith('restricted/')):1417 cache[pkg].candidate.section.startswith(
1418 'restricted/')):
1418 difference.add(pkg)1419 difference.add(pkg)
1419 del cache1420 del cache
14201421

Subscribers

People subscribed via source and target branches