Ubuntu
pkg-create-dbgsym package

pkg-create-dbgsym parsing of debian/control doesn't match dpkg's

Bug #627122 reported by Steve Langasek
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
pkg-create-dbgsym (Ubuntu)
Fix Released
Low
Steve Langasek

Bug Description

Binary package hint: pkg-create-dbgsym

emdebian-crush failed to build because pkg-create-dbgsym wasn't happy with debian/control. This wasn't caught pre-upload because dpkg-dev itself *is* happy with debian/control.

http://launchpadlibrarian.net/54444678/buildlog_ubuntu-maverick-i386.emdebian-crush_2.2.4ubuntu1_FAILEDTOBUILD.txt.gz

The cause of this failure was a missing space after the field name in "Package:pdebuild-cross".

Although it's best practice to include the space, pkg-create-dbgsym shouldn't fail on debian/control files that dpkg considers valid.

Tags: patch
Steve Langasek (vorlon)
Changed in pkg-create-dbgsym (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Wookey (wookey) wrote :

It seems emdebian-crush isn't the first package to fall foul of this problem: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=460959 suggests that it has been around for at least 2.5 years.

So here is a patch to fix it. It might have been a bit less cryptic in awk, but this is quicker.

Brian Murray (brian-murray)
tags: added: patch
Loïc Minier (lool)
Changed in pkg-create-dbgsym (Ubuntu):
assignee: nobody → Steve Langasek (vorlon)
status: Triaged → Fix Released
status: Fix Released → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pkg-create-dbgsym - 0.44

---------------
pkg-create-dbgsym (0.44) maverick; urgency=low

  * dh_strip: debian/control fields aren't required to have a space after
    the field name; fix the parsing code to still be able to detect
    package/architecture fields correctly in this case. Thanks to
    Wookey <email address hidden> for the initial patch. LP: #627122.
  * tests/dhtest.whitespace: extend test case with more interesting
    whitespace usage for Package/Architecture fields, to match.
 -- Steve Langasek <email address hidden> Thu, 02 Sep 2010 16:01:02 +0000

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

Other bug subscribers

Remote bug watches

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