[submittodebian] fails with "Version number suggests Ubuntu changes, but Maintainer: does not have Ubuntu address" unless DEBEMAIL overridden

Bug #1015066 reported by James Page
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

I noticed the change in behaviour between precise->quantal with regards to building the package from source to generate the debdiff; I see this error unless I override DEBEMAIL to not be an @ubuntu.com address.

jamespage@hendrix:~/src/ubuntu/quantal/tiles$ submittodebian
Building using working tree
Building package in normal mode
Looking for a way to retrieve the upstream tarball
Using the upstream tarball that is present in /home/jamespage/src/ubuntu/quantal
Building the package in /home/jamespage/src/ubuntu/quantal/build-area/tiles-2.2.2, using debuild -uc -us -nc -S
 dpkg-buildpackage -rfakeroot -d -us -uc -sa -nc -S
dpkg-buildpackage: source package tiles
dpkg-buildpackage: source version 2.2.2-2ubuntu2
dpkg-buildpackage: source changed by James Page <email address hidden>
 dpkg-source --before-build tiles-2.2.2
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 tiles-2.2.2
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 tiles-2.2.2 gave error exit status 255
debuild: fatal error at line 1357:
dpkg-buildpackage -rfakeroot -d -us -uc -sa -nc -S failed
bzr: ERROR: The build failed.
Traceback (most recent call last):
  File "/usr/bin/submittodebian", line 243, in <module>
    main()
  File "/usr/bin/submittodebian", line 226, in main
    build_source_package()
  File "/usr/bin/submittodebian", line 74, in build_source_package
    check_call(cmd)
  File "/usr/lib/python2.7/dist-packages/ubuntutools/subprocess.py", line 77, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['bzr', 'bd', '-S', '--', '-uc', '-us', '-nc']' returned non-zero exit status 3

Not sure what the rationale for changing this was but it takes alot longer to generate the debdiff from a bzr branch than it used to.

ProblemType: Bug
DistroRelease: Ubuntu 12.10
Package: ubuntu-dev-tools 0.142
ProcVersionSignature: Ubuntu 3.4.0-5.11-generic 3.4.0
Uname: Linux 3.4.0-5-generic x86_64
NonfreeKernelModules: fglrx
ApportVersion: 2.2.3-0ubuntu4
Architecture: amd64
Date: Tue Jun 19 11:42:14 2012
PackageArchitecture: all
SourcePackage: ubuntu-dev-tools
UpgradeStatus: Upgraded to quantal on 2012-06-11 (8 days ago)

Revision history for this message
James Page (james-page) wrote :
Revision history for this message
Stefano Rivera (stefanor) wrote :

> Not sure what the rationale for changing this was

So that we could reverse update-maintainer before building the source package.

This sounds like a relative of LP: #1007042. I'll take the same approach.

Changed in ubuntu-dev-tools (Ubuntu):
status: New → 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.

Other bug subscribers

Remote bug watches

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