Merge ~jibel/ubiquity:remove_popcon into ubiquity:master

Proposed by Jean-Baptiste Lallement
Status: Merged
Merged at revision: c1621984dafb405b0284303111962c35cc1480c8
Proposed branch: ~jibel/ubiquity:remove_popcon
Merge into: ubiquity:master
Diff against target: 62 lines (+7/-12)
3 files modified
debian/changelog (+7/-1)
scripts/plugininstall.py (+0/-10)
ubiquity/i18n.py (+0/-1)
Reviewer Review Type Date Requested Status
Iain Lane Approve
Review via email: mp+390882@code.launchpad.net

Commit message

popularity-contest is not shipped anymore, remove it from the installer.

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

I'm good with this, but I'm guessing since we don't ship popcon anywhere currently we were always getting into the exception handler already?

Previously any flavour could enable popcon by seeding it, this will break that, but I think it's OK since the server is not maintained.

Please pull --rebase before pushing. I can't do that at my end because that will mean LP loses track of the merge.

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

Thanks for the review.

Yes, that was the case, and on Ubuntu it generates systematically an error in the logs, thats how I noticed it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 9049fe9..577573f 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,9 @@
6+ubiquity (20.10.11) UNRELEASED; urgency=medium
7+
8+ * Remove popularity-contest.
9+
10+ -- Jean-Baptiste Lallement <jean-baptiste.lallement@ubuntu.com> Thu, 17 Sep 2020 08:00:35 +0200
11+
12 ubiquity (20.10.10) groovy; urgency=medium
13
14 [ Didier Roche ]
15@@ -5,7 +11,7 @@ ubiquity (20.10.10) groovy; urgency=medium
16 * AD integration for GTK UI.
17 * bump KDE_SESSION_VERSION to 5 to fix xdg-open on Kubuntu (LP: #1890173)
18
19- [ Brian Murray ]
20+ [Brian Murray]
21 * Do not try to keep installed software when reinstalling Ubuntu as the
22 process was error prone (LP: 1152399) and did not provide a quailty
23 experience. (LP: #1798369)
24diff --git a/scripts/plugininstall.py b/scripts/plugininstall.py
25index 48e7491..acfe9d0 100755
26--- a/scripts/plugininstall.py
27+++ b/scripts/plugininstall.py
28@@ -784,15 +784,6 @@ class Install(install_misc.InstallBase):
29 script += '-oem'
30 misc.execute(script)
31
32- osextras.unlink_force(self.target_file('etc/popularity-contest.conf'))
33- try:
34- participate = self.db.get('popularity-contest/participate')
35- install_misc.set_debconf(
36- self.target, 'popularity-contest/participate', participate,
37- self.db)
38- except debconf.DebconfError:
39- pass
40-
41 osextras.unlink_force(self.target_file('etc/papersize'))
42 subprocess.call(['log-output', '-t', 'ubiquity', 'chroot', self.target,
43 'ucf', '--purge', '/etc/papersize'],
44@@ -824,7 +815,6 @@ class Install(install_misc.InstallBase):
45 pass
46
47 packages = ['linux-image-' + self.kernel_version,
48- 'popularity-contest',
49 'libpaper1',
50 'ssl-cert']
51 arch, subarch = install_misc.archdetect()
52diff --git a/ubiquity/i18n.py b/ubiquity/i18n.py
53index 8b6bf6e..d7f6555 100644
54--- a/ubiquity/i18n.py
55+++ b/ubiquity/i18n.py
56@@ -102,7 +102,6 @@ def get_translations(languages=None, core_names=[], extra_prefixes=[]):
57 'partman-target/no_root',
58 'partman-target/text/method',
59 'grub-installer/bootdev',
60- 'popularity-contest/participate',
61 ))
62 prefixes = reduce(lambda x, y: x + '|' + y, extra_prefixes, prefixes)
63

Subscribers

People subscribed via source and target branches