pbuilder-satisfydepends breaks on comments

Bug #1632980 reported by Dmitry
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pbuilder (Debian)
Fix Released
Unknown
pbuilder (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

I cannot rebuild pulseaudio with pdebuild because pbuilder-satisfydepends returns an empty list, and pdebuild fails with unsatisfied dependencies. See https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1632478 . Pulseaudio control file contains comments and pbuilder-satisfydepends fails to parse them.

ProblemType: Bug
DistroRelease: Ubuntu 16.04
Package: pbuilder 0.223
ProcVersionSignature: Ubuntu 4.4.0-38.57-generic 4.4.19
Uname: Linux 4.4.0-38-generic i686
ApportVersion: 2.20.1-0ubuntu2.1
Architecture: i386
CurrentDesktop: KDE
Date: Thu Oct 13 10:49:57 2016
InstallationDate: Installed on 2016-08-15 (58 days ago)
InstallationMedia: Kubuntu 16.04 LTS "Xenial Xerus" - Release i386 (20160420.1)
PackageArchitecture: all
SourcePackage: pbuilder
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Dmitry (dmitrymq) wrote :
Revision history for this message
Jessica Clarke (jrtc27) wrote :

As far as I can see, the only comments involving Build-Depends for pulseaudio 8.0-0ubuntu3 are the following, *before* Build-Depends:

# The following packages can be omitted for bootstrapping (DEB_BUILD_PROFILES=stage1):
# libbluetooth-dev

Please post the full build log, otherwise we can't help you.

Changed in pbuilder (Ubuntu):
status: New → Incomplete
Revision history for this message
Dmitry (dmitrymq) wrote :

The full build log is attached

Revision history for this message
Mattia Rizzolo (mapreri) wrote :

I can't reproduce it with 0.226.1.
And apparently I made so many changes that I can't easily try to run 0.223 with my .pbuilderrc that is taking advantage of some later changes (!) and I'm way too lazy revert those changes...

Could you please try with a newer pbuilder? What you can get from yakkety repository is pretty ok (you can just install the .deb, there is nothing yakkety-specific).

Revision history for this message
Dmitry (dmitrymq) wrote :

Mattia Rizzolo (mapreri)

>And apparently I made so many changes that I can't easily try to run 0.223

You can use Ubuntu Live Cd without installing anything. With Virtualbox you don't need even to reboot.

Jessica Clarke (jrtc27)
Changed in pbuilder (Ubuntu):
status: Incomplete → Fix Committed
Changed in pbuilder (Debian):
status: Unknown → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.4 KiB)

This bug was fixed in the package pbuilder - 0.228

---------------
pbuilder (0.228) unstable; urgency=medium

  [ Mattia Rizzolo ]
  * modules:
    + Use default value of I if LOGLEVEL is undefined (fixes missing output
      from pbuilder-satisfydepends-apt).
    + create_basetgz(): explicitly exclude /sys/* and /proc/* from being saved
      in the tarball. Closes: #560332
    + mount a tmpfs on /dev/shm by default, and set up a symlink
      /run/shm → /dev/shm.
      This also renames the configuration variable USERUNSHM to USESHM
      (deprecating the former).
      This will happen regardless of the Debian version of the chroot and the
      architecture used, as it ought to be harmless if unused. Administrators
      are suggested to drop any workarounds in place to overcome this
      deficiency in pbuilder, such as bind-mounting /dev/shm from the host, to
      avoid situations like over-mounting.
      Closes: #591291, #717022, #773765
    + cross_build_setup(): run `apt-get update` after adding the foreign
      architecture in dpkg. Otherwise apt won't know a thing about the
      foreign architectures, and won't be able to install anything.
    + extractbuildplace(): if ARCHITECTURE is not set, finalize it and
      HOST_ARCH right after chroot extraction, using the native architecture
      of the chroot.
  * pbuilder.8:
    + Fix a typo and add a missing word.
      Thanks to Unit193 for spotting and reporting.
  * pbuilderrc.5:
    + correct default for CCACHEDIR, which is not enabled by default.
  * satifydepends-apt:
    + Use the new --indep-only flag from APT 1.4~beta3 when performing an
      indep-only build. This bumps the required apt version inside the chroot.
  * d/NEWS:
    + Add news entry for the SHM behaviour change.
  * t:
    + testlib: Autodetect if this is running from a git checkout and
      automatically set PBUILDER_CHECKOUT.

  [ James Clarke ]
  * modules:
    + add a get822field() function.
  * buildpackage-funcs:
    + add a dsc_get_basename() function returning the standard
      $srcname-$version as expected by other tools.
  * satisfydepends-funcs:
    + split get_build_depends_field() out of get_source_control_field().
      get_source_control_field() is now only a wrapper around the new
      get822field(), whilst the logic to clean up and normalize the dependency
      list is moved into get_build_depends_field(). Change get_build_deps()
      and get_build_conflicts() to use this new function.
    + Make get_source_control_field() more robust.
      This includes fixing comments or blank lines just before a field causing
      the field to be ignored. Closes: #847944; LP: #1632980
  * Add new --source-only-changes option, and related SOURCE_ONLY_CHANGES
    config variable, to ask pbuilder to also produce a _sources.changes file,
    suitable for a source only upload.
  * pdebuild:
    + Use dpkg-source instead of dpkg-buildpackage to build the source package.
      This will avoid generating a .changes that can't be used in the case
      the source package is rebuilt (as the rebuilt source package is not
      guaranteed to be the same as the original one). Although, if
      BUILDRES...

Read more...

Changed in pbuilder (Ubuntu):
status: Fix Committed → Fix Released
Changed in pbuilder (Debian):
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.