config-package-dev 5.2 broke transforms
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
config-package-dev (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Artful |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
As reported at https:/
Can't use string ("/ARRAY(
config-package-dev is a tool used as a build-dependency in private (third-party) Debian packages, so this causes those packages to FTBFS in artful when they built successfully in older releases.
Bruno Maitre reported the problem and provided a patch to fix the bug, which has been incorporated into version 5.3 in Debian.
[Test Case]
config-package-dev ships one example that uses the transform functionality, but the version in 5.2 doesn't quite build right in artful (or in Debian unstable). So the easiest way is to grab the updated example from git:
apt install config-package-dev debhelper lynx
wget http://
tar xf master.tar.gz
cd config-
dpkg-buildpackage
With config-package-dev 5.2, this will FTBFS at the "dh_configpackage" step; with the fixed version, the package will build. (If you're curious you can try dpkg -i'ing the resulting package, which will change lynx's home page to default to web.mit.edu.)
[Regression Potential]
I'm attaching two debdiffs to this bug. One backports version 5.4 from Debian unstable, which in addition to fixing this bug, fixes the examples to build on unstable, adds an autopkgtest to make sure the examples build, and makes no other code changes. The other just cherry-picks the one change.
Arguably the regression potential for the autopkgtest version is less, since if tests pass that gives us confidence that the new version works on Ubuntu. (I have been doing my testing on Debian.) I'd prefer this version, under the rules for SRUing an upstream microrelease of a package with a test suite. But it does change more; if you prefer the targeted fix that's fine with me too.
The regression potential compared to 5.2 is extremely low, since 5.2 is definitely broken for packages that use transforms, and Bruno's patch only affects the code that handles transforms. I am also confident in the patch now that I have added autopkgtests and they pass on Debian unstable, stretch, and jessie (thanks, travis.
Also, as noted the previous time we SRU'd config-package-dev (LP #899732), nothing in the Ubuntu or Debian archive build-depends on config-package-dev; it's a tool used by external packages. So even if there is a regression it will be low-severity.
description: | updated |
Changed in config-package-dev (Ubuntu): | |
status: | Fix Committed → Fix Released |
Changed in config-package-dev (Ubuntu Artful): | |
status: | Confirmed → In Progress |
assignee: | nobody → Nish Aravamudan (nacc) |
Above is a backport of the patch (a two-line fix to some Perl) to 5.2.
This is a backport of the entire 5.4 version, which includes autopkgtests and fixes the examples to successfully build. The diff might be easier to read on GitHub: https:/ /github. com/sipb/ config- package- dev/compare/ 5.2...5. 4
(I can provide these two as Git branches if that's easier for anyone)