[backportpackage] errors out on packages with no ubuntu revision

Bug #1007042 reported by Jeremy Bícha
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

This worked last week, but now fails. I'm filing against ubuntu-dev-tools since I know that just got an update this week.

It also fails if I use -d unstable.

$ backportpackage -s quantal -d precise --suffix= -u ppa:texlive-backports/ppa texlive-bin
backportpackage: Downloading texlive-bin_2012.20120530.orig.tar.xz from archive.ubuntu.com (29.769 MiB)
backportpackage: Downloading texlive-bin_2012.20120530-2.debian.tar.gz from archive.ubuntu.com (0.063 MiB)
dpkg-source: info: extracting texlive-bin in texlive-bin-precise
dpkg-source: info: unpacking texlive-bin_2012.20120530.orig.tar.xz
dpkg-source: info: unpacking texlive-bin_2012.20120530-2.debian.tar.gz
dch warning: new version (2012.20120530-2~ubuntu12.04.1) is less than
the current version number (2012.20120530-2).
 dpkg-buildpackage -rfakeroot -d -us -uc -S -nc -sd -v2009-11ubuntu2
dpkg-buildpackage: source package texlive-bin
dpkg-buildpackage: source version 2012.20120530-2~ubuntu12.04.1
dpkg-buildpackage: source changed by Jeremy Bicha <EMAIL HIDDEN>
 dpkg-source --before-build texlive-bin-precise
dpkg-buildpackage: warning: it is a bad idea to generate a source package without cleaning up first, it might contain undesired files.
 dpkg-source -b texlive-bin-precise
dpkg-source: error: Version number suggests Ubuntu changes, but Maintainer: does not have Ubuntu address
dpkg-source: info: using source format `3.0 (quilt)'
dpkg-buildpackage: error: dpkg-source -b texlive-bin-precise gave error exit status 255
debuild: fatal error at line 1350:
dpkg-buildpackage -rfakeroot -d -us -uc -S -nc -sd -v2009-11ubuntu2 failed
backportpackage: Error: debuild returned 29.

ProblemType: Bug
DistroRelease: Ubuntu 12.10
Package: ubuntu-dev-tools 0.142
ProcVersionSignature: Ubuntu 3.4.0-3.7-generic 3.4.0
Uname: Linux 3.4.0-3-generic x86_64
ApportVersion: 2.1.1-0ubuntu1
Architecture: amd64
CheckboxSubmission: 201084ed41ea7548f8becd47fa4d1e6c
CheckboxSystem: c541d13ea4f205f2fd751f76ed21105b
Date: Thu May 31 13:56:28 2012
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Release amd64 (20120425)
PackageArchitecture: all
SourcePackage: ubuntu-dev-tools
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Jeremy Bícha (jbicha) wrote :
Revision history for this message
Stefano Rivera (stefanor) wrote : Re: [Bug 1007042] [NEW] [backportpackage] fails but worked last week

 subscribe ubuntu-backporters

Hi Jeremy (2012.05.31_20:04:33_+0200)
> This worked last week, but now fails. I'm filing against ubuntu-dev-
> tools since I know that just got an update this week.

It's fallout from the backport version schema change.

The versions are now ~ubuntu12.04.1 instead of ~precise1 (as code names
are due to wrap in the foreseeable future)

That matches some code in the Ubuntu Vendor plugin to dpkg-source that
looks for "ubuntu" in the version without @ubuntu.com in the Maintainer.

This can be lowered from an error to a warning by un-exporting DEBEMAIL.

Or we could update the maintainer field of backports. But that's up to
the backporters.

Revision history for this message
Benjamin Drung (bdrung) wrote : Re: [backportpackage] fails but worked last week

Wouldn't it be better to tweak dpkg-source to look for "ubuntu", but not for "~ubuntu"?

Revision history for this message
Jeremy Bícha (jbicha) wrote :

By the way, what does this mean?

"backportpackage: Switch to ~ubuntu12.04.1-style version numbers instead of ~precise1, to make our version numbers more future-proof."

Revision history for this message
Evan Broder (broder) wrote :

We used to generate backport version numbers of the form VERSION_BEING_BACKPORTED~RELEASE_CODENAME1. It worked well at the time because it guaranteed that a backport of the same version to, e.g., natty would have a lower version number than the backport to oneiric, since the n would sort before o. That's important because we rely on the version numbers sorting like that for release upgrade paths with backports to work.

Colin pointed out that this would break down when we ran out of letters and started wrapping around, and suggested that we instead switch to using the version number, which is what backportpackage now does.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu-dev-tools (Ubuntu):
status: New → Confirmed
Micah Gersten (micahg)
summary: - [backportpackage] fails but worked last week
+ [backportpackage] errors out on packages with no ubuntu revision
Benjamin Drung (bdrung)
Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Medium
Changed in ubuntu-dev-tools (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-dev-tools - 0.143

---------------
ubuntu-dev-tools (0.143) unstable; urgency=low

  [ Iain Lane ]
  * backportpackage: Fix filenames searched when looking for existing
    .orig.tar.foo files (to determine if we need to upload it again or not).
    (LP: #1007908)
  * backportpackage: Unset DEBEMAIL when building source package. Fixes error
    when building backports for packages with no Ubuntu changes.
    (LP: #1007042)

  [ Mathieu Trudel-Lapierre ]
  * mk-sbuild: use and update messages to suggest using the source:$chroot way
    of referring to source chroots instead of $chroot-source; since the latter
    does not work with btrfs snapshot-based chroots. (LP: #1014669)

  [ Lars Düsing ]
  * Corrected brackets in man-page for sponsor-patch.

  [ Stefano Rivera ]
  * pbuilder-dist: Don't try to enable -updates for the current codename
    referring to Debian testing, either (LP: #1011870)
  * Correct spelling mistakes in package description, thanks Logan Rosen for
    the patch (Closes: #678245)
  * Correct metavar for --mirror in backportpackage (LP: #999727)
  * submittodebian: Explitictly UTF-8 encode the bug body (LP: #1005834)
  * backportpackage.1: Document --key and --dont-sign (LP: #1007564)
  * seeded-in-ubuntu: Catch errors in parsing data, and don't keep unreadable
    data cached (LP: #1008783)
  * ubuntutools.archive: Improve error handling around rmadison calls
    (LP: #1010951)
  * submittodebian; Unset DEBEMAIL when building source package.
    (LP: #1015066)

  [ Benjamin Drung ]
  * Add a man page for the reverse-build-depends wrapper script.

 -- Benjamin Drung <email address hidden> Fri, 22 Jun 2012 13:34:43 +0200

Changed in ubuntu-dev-tools (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.