Merge lp:~laney/ubiquity/lp1715605 into lp:ubiquity

Proposed by Iain Lane
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 6563
Merged at revision: 6563
Proposed branch: lp:~laney/ubiquity/lp1715605
Merge into: lp:ubiquity
Diff against target: 28 lines (+11/-0)
2 files modified
debian/changelog (+9/-0)
ubiquity/plugins/ubi-console-setup.py (+2/-0)
To merge this branch: bzr merge lp:~laney/ubiquity/lp1715605
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
Review via email: mp+330352@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks good. I think this is probably the best way to avoid the maintscript-helper running. Those were never designed to be run through ubiquity / outside of a proper package install scenario, so we need to accomodate.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2017-09-06 14:30:53 +0000
+++ debian/changelog 2017-09-07 12:19:34 +0000
@@ -1,3 +1,12 @@
1ubiquity (17.10.7) UNRELEASED; urgency=medium
2
3 * ubi-console-setup.py: Set DPKG_MAINTSCRIPT_{NAME,PACKAGE} variables for
4 our manual running of keyboard-configuration.postinst. This script now
5 runs dpkg-maintscript-helper to remove an old conffile. That code won't
6 fire in ubiquity, but we still need to not error out. (LP: #1715605)
7
8 -- Iain Lane <iain.lane@canonical.com> Thu, 07 Sep 2017 13:15:32 +0100
9
1ubiquity (17.10.6) artful; urgency=medium10ubiquity (17.10.6) artful; urgency=medium
211
3 [ David Mohammed ]12 [ David Mohammed ]
413
=== modified file 'ubiquity/plugins/ubi-console-setup.py'
--- ubiquity/plugins/ubi-console-setup.py 2017-07-18 15:21:41 +0000
+++ ubiquity/plugins/ubi-console-setup.py 2017-09-07 12:19:34 +0000
@@ -477,6 +477,8 @@
477 'OVERRIDE_USE_DEBCONF_LOCALE': '1',477 'OVERRIDE_USE_DEBCONF_LOCALE': '1',
478 'LC_ALL': di_locale,478 'LC_ALL': di_locale,
479 'PATH': '/usr/lib/ubiquity/console-setup:' + os.environ['PATH'],479 'PATH': '/usr/lib/ubiquity/console-setup:' + os.environ['PATH'],
480 'DPKG_MAINTSCRIPT_NAME': 'postinst',
481 'DPKG_MAINTSCRIPT_PACKAGE': 'keyboard-configuration',
480 }482 }
481 return command, questions, environ483 return command, questions, environ
482484

Subscribers

People subscribed via source and target branches

to status/vote changes: