Debian Bug report logs - #586275
sbuild: alternative build-deps and Provides do not work correctly

Toggle useless messages

Message #3 received at submit@bugs.debian.org (full text, mbox, reply):

From: Ansgar Burchardt <ansgar@2008.43-1.org>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: sbuild: alternative build-deps and Provides do not work correctly

Date: Fri, 18 Jun 2010 11:50:04 +0900

Package: sbuild
Version: 0.60.0-1
Severity: normal

Hi,

sbuild does not handle alternative dependencies that are also Provided
by another package correctly:

libcpan-meta-perl_2.101670-1 has a build-dependency on
libversion-perl (>= 1:0.8200) | perl (>= 5.12).  libversion-perl is also
provided by perl-modules.

sbuild fails to resolve this dependency:

  libversion-perl: non-matching version installed (~*=PROVIDED=*= ! >= 1:0.8200)
  Using default version 1:0.8200-1
  perl: non-matching version installed (5.10.1-13 ! >= 5.12)
  Default version of perl not sufficient, Package installation not possible
  Source-dependencies not satisfied; skipping libcpan-meta-perl

Somehow it looks like it wants to install both alternatives?

When the alternative on perl (>= 5.12) is removed, sbuild builds the
package just fine.

Regards,
Ansgar

-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (900, 'testing'), (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/2 CPU cores)
Locale: LANG=C, LC_CTYPE=ja_JP.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages sbuild depends on:
ii  adduser                       3.112      add and remove users and groups
ii  libsbuild-perl                0.60.0-1   Tool for building Debian binary pa
ii  perl                          5.10.1-13  Larry Wall's Practical Extraction 
ii  perl-modules                  5.10.1-13  Core Perl modules

Versions of packages sbuild recommends:
ii  debootstrap                   1.0.23     Bootstrap a basic Debian system
ii  fakeroot                      1.14.4-1   Gives a fake root environment

Versions of packages sbuild suggests:
pn  deborphan                     <none>     (no description available)
ii  wget                          1.12-2     retrieves files from the web

-- Configuration Files:
/etc/sbuild/nssdatabases-defaults changed [not included]

-- no debconf information

Message #8 received at 586275@bugs.debian.org (full text, mbox, reply):

From: Roger Leigh <rleigh@codelibre.net>

To: Ansgar Burchardt <ansgar@2008.43-1.org>, 586275@bugs.debian.org

Subject: Re: [buildd-tools-devel] Bug#586275: sbuild: alternative build-deps and Provides do not work correctly

Date: Fri, 18 Jun 2010 15:20:07 +0100

[Message part 1 (text/plain, inline)]

On Fri, Jun 18, 2010 at 11:50:04AM +0900, Ansgar Burchardt wrote:
> sbuild does not handle alternative dependencies that are also Provided
> by another package correctly:
> 
> libcpan-meta-perl_2.101670-1 has a build-dependency on
> libversion-perl (>= 1:0.8200) | perl (>= 5.12).  libversion-perl is also
> provided by perl-modules.

I'm afraid this is a long-standing issue with sbuild.  Alternative
build-dependencies are just not supported.  You'll see there are
a number of existing bug reports regarding this issue.

Note that one disadvantage of them is that it can potentially make
builds non-deterministic, which is probably why not much effort
went in to making them work.

However, the sbuild dependency resolver is now modular, so you can
use an aptitude-based resolver in place of the older internal
resolver, which /may/ work better (but isn't in use on the normal
buildds yet).

Just set $build_dep_resolver="internal|aptitude" in sbuild.conf.


Regards,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.

[signature.asc (application/pgp-signature, inline)]

Message #13 received at 586275@bugs.debian.org (full text, mbox, reply):

From: Ansgar Burchardt <ansgar@43-1.org>

To: Roger Leigh <rleigh@codelibre.net>

Cc: 586275@bugs.debian.org

Subject: Re: [buildd-tools-devel] Bug#586275: sbuild: alternative build-deps and Provides do not work correctly

Date: Sat, 19 Jun 2010 08:55:00 +0900

Roger Leigh <rleigh@codelibre.net> writes:

> On Fri, Jun 18, 2010 at 11:50:04AM +0900, Ansgar Burchardt wrote:
>> sbuild does not handle alternative dependencies that are also Provided
>> by another package correctly:
>> 
>> libcpan-meta-perl_2.101670-1 has a build-dependency on
>> libversion-perl (>= 1:0.8200) | perl (>= 5.12).  libversion-perl is also
>> provided by perl-modules.
>
> I'm afraid this is a long-standing issue with sbuild.  Alternative
> build-dependencies are just not supported.  You'll see there are
> a number of existing bug reports regarding this issue.
>
> Note that one disadvantage of them is that it can potentially make
> builds non-deterministic, which is probably why not much effort
> went in to making them work.

As far as I know, sbuild's internal resolver tries to satisfy the first
alternative.  But the problem in this case is caused by sbuild trying to
satisfy *both* alternative dependencies instead of ignoring everything
but the first alternative.

sbuild should only install libversion-perl instead of both
libversion-perl and perl 5.12.

Regards,
Ansgar

Message #18 received at 586275@bugs.debian.org (full text, mbox, reply):

From: Ansgar Burchardt <ansgar@43-1.org>

To: 586275@bugs.debian.org

Subject: Re: sbuild: alternative build-deps and Provides do not work correctly

Date: Mon, 05 Jul 2010 17:47:47 +0900

[Message part 1 (text/plain, inline)]

tags 586275 + patch
thanks

I prepared a patch for this bug which seems to work at least for my
problem:

1. Grab libdatetime-perl_2:0.5900-1 from the archive.
2. Change
     libmodule-build-perl (>= 0.360100)
   to
     libmodule-build-perl (>= 0.360100) | perl (>= 5.12)
   in the Build-Depends field in debian/control.
3. Try to build the package with sbuild's internal resolver.

Expected behavior:
sbuild will be able to build the package and resolve dependencies in the
same way.

Observed behavior:
Building the package will abort with the following error message:

  perl: already installed (5.10.1-13)
  debhelper: missing
  Using default version 7.9.3
  libmodule-build-perl: non-matching version installed (~*=PROVIDED=*= ! >= 0.360100)
  Using default version 0.360700-1
  perl: non-matching version installed (5.10.1-13 ! >= 5.12)
  Default version of perl not sufficient, Package installation not possible
  Source-dependencies not satisfied; skipping libdatetime-perl

After the patch is applied, sbuild is happy to only install the default
version of libmodule-build-perl and ignore the alternative dependency on
perl:

  perl: already installed (5.10.1-13)
  debhelper: missing
  Using default version 7.9.3
  libmodule-build-perl: non-matching version installed (~*=PROVIDED=*= ! >= 0.360100)
  Using default version 0.360700-1
  perl: already installed (5.10.1-13 >= 5.10.1 is satisfied)
  libparams-validate-perl: missing
  libdatetime-locale-perl: missing
  Using default version 1:0.45-1
  libdatetime-timezone-perl: missing
  Using default version 1:1.19-1+2010j
  libtest-exception-perl: missing
  libtest-warn-perl: missing

Note that I have not looked at the code in all detail and thus the patch
might break in other cases.

Regards,
Ansgar

[0001-Do-not-try-other-alternatives-if-upgrading-is-enough.patch (text/x-diff, attachment)]

Message #25 received at 586275@bugs.debian.org (full text, mbox, reply):

From: gregor herrmann <gregoa@debian.org>

To: 591189@bugs.debian.org, 591189-submitter@bugs.debian.org, 591187@bugs.debian.org, 591187-submitter@bugs.debian.org, 591188@bugs.debian.org, 591188-submitter@bugs.debian.org, 586275@bugs.debian.org

Cc: control@bugs.debian.org

Subject: Actually an sbuild problem

Date: Sun, 1 Aug 2010 17:44:18 -0400

[Message part 1 (text/plain, inline)]

reassign 591189 sbuild
affects  591189 libcpan-meta-perl
reassign 591187 sbuild
affects  591187 libmoosex-types-perl-perl
reassign 591188 sbuild
affects  591188 libnetpacket-perl
merge 586275 591189 591187 591188
thanks

These three bugs are actually duplicates of 586275.

Of course we can change the dependencies, but IMO they are correct
and it's sbuild having issues. Hopefully the patch in 586275 can be
applied (or commented on) soon.

I'm happy to discuss other ways of dealing with this bugs if someone
thinks this is not correct.

Cheers,
gregor

-- 
 .''`.   http://info.comodo.priv.at/ -- GPG key IDs: 0x8649AA06, 0x00F3CFE4
 : :' :  Debian GNU/Linux user, admin, & developer - http://www.debian.org/
 `. `'   Member of VIBE!AT & SPI, fellow of Free Software Foundation Europe
   `-    BOFH excuse #423:  It's not RFC-822 compliant. 

[signature.asc (application/pgp-signature, inline)]

Message #42 received at 586275@bugs.debian.org (full text, mbox, reply):

From: Ansgar Burchardt <ansgar@43-1.org>

To: 586275@bugs.debian.org, 588035@bugs.debian.org

Subject: Status of these two patches

Date: Sun, 15 Aug 2010 23:06:33 +0900

Hi,

it's been a month since I provided patches for #586275 and #588035.  As
they are not yet applied in the Git repository, I wonder if there's any
problem with them.

Regards,
Ansgar

Message #49 received at 586275@bugs.debian.org (full text, mbox, reply):

From: Roger Leigh <rleigh@codelibre.net>

To: Ansgar Burchardt <ansgar@43-1.org>, 586275@bugs.debian.org

Cc: 588035@bugs.debian.org

Subject: Re: [buildd-tools-devel] Bug#586275: Status of these two patches

Date: Sun, 15 Aug 2010 15:39:45 +0100

[Message part 1 (text/plain, inline)]

On Sun, Aug 15, 2010 at 11:06:33PM +0900, Ansgar Burchardt wrote:
> it's been a month since I provided patches for #586275 and #588035.  As
> they are not yet applied in the Git repository, I wonder if there's any
> problem with them.

Not any problem with the patches (that I've seen).  I'm currently
lacking the time to review and apply them.  I'm afraid I haven't
had time for sbuild work for at least a month.  I'll see if I
can find time this evening, but I can't make any promises.

If anyone else can do this, I'd much appreciate it.


Regards,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.

[signature.asc (application/pgp-signature, inline)]

Message #58 received at 586275@bugs.debian.org (full text, mbox, reply):

From: Roger Leigh <rleigh@codelibre.net>

To: Ansgar Burchardt <ansgar@43-1.org>

Cc: 586275@bugs.debian.org

Subject: Re: sbuild: alternative build-deps and Provides do not work correctly

Date: Thu, 28 Oct 2010 22:05:55 +0100

[Message part 1 (text/plain, inline)]

tags 586275 + fixed-upstream pending
thanks

On Mon, Jul 05, 2010 at 05:47:47PM +0900, Ansgar Burchardt wrote:
> I prepared a patch for this bug which seems to work at least for my
> problem:
> 
> 1. Grab libdatetime-perl_2:0.5900-1 from the archive.
> 2. Change
>      libmodule-build-perl (>= 0.360100)
>    to
>      libmodule-build-perl (>= 0.360100) | perl (>= 5.12)
>    in the Build-Depends field in debian/control.
> 3. Try to build the package with sbuild's internal resolver.
> 
> Expected behavior:
> sbuild will be able to build the package and resolve dependencies in the
> same way.

I've applied your patch to the sbuild master branch.
I'm not sure of the best way to test whether it will work correctly
in all situations, so we'll have to rely on people reporting bugs
if this has any negative effects.


Regards,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.

[signature.asc (application/pgp-signature, inline)]

Message #65 received at 586275-close@bugs.debian.org (full text, mbox, reply):

From: Roger Leigh <rleigh@debian.org>

To: 586275-close@bugs.debian.org

Subject: Bug#586275: fixed in sbuild 0.60.2-1

Date: Tue, 02 Nov 2010 15:17:14 +0000

Source: sbuild
Source-Version: 0.60.2-1

We believe that the bug you reported is fixed in the latest version of
sbuild, which is due to be installed in the Debian FTP archive:

buildd_0.60.2-1_all.deb
  to main/s/sbuild/buildd_0.60.2-1_all.deb
libsbuild-perl_0.60.2-1_all.deb
  to main/s/sbuild/libsbuild-perl_0.60.2-1_all.deb
sbuild_0.60.2-1.diff.gz
  to main/s/sbuild/sbuild_0.60.2-1.diff.gz
sbuild_0.60.2-1.dsc
  to main/s/sbuild/sbuild_0.60.2-1.dsc
sbuild_0.60.2-1_all.deb
  to main/s/sbuild/sbuild_0.60.2-1_all.deb
sbuild_0.60.2.orig.tar.gz
  to main/s/sbuild/sbuild_0.60.2.orig.tar.gz
wanna-build_0.60.2-1_all.deb
  to main/s/sbuild/wanna-build_0.60.2-1_all.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 586275@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Roger Leigh <rleigh@debian.org> (supplier of updated sbuild package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: RIPEMD160

Format: 1.8
Date: Tue, 02 Nov 2010 13:02:50 +0000
Source: sbuild
Binary: libsbuild-perl sbuild wanna-build buildd
Architecture: source all
Version: 0.60.2-1
Distribution: unstable
Urgency: low
Maintainer: Debian buildd-tools Developers <buildd-tools-devel@lists.alioth.debian.org>
Changed-By: Roger Leigh <rleigh@debian.org>
Description: 
 buildd     - Daemon for automatically building Debian binary packages from Deb
 libsbuild-perl - Tool for building Debian binary packages from Debian sources
 sbuild     - Tool for building Debian binary packages from Debian sources
 wanna-build - Database to track building of Debian binary packages from Debian
Closes: 380113 567506 567821 567822 569823 571806 578810 579971 586275 588035 590153 591187 591188 591189 595863 595866 595872 595934 598054 599348 599350
Changes: 
 sbuild (0.60.2-1) unstable; urgency=low
 .
   * New release.
   * sbuild:
     - Dependency resolving:
       . The aptitude build dependency resolver now sets
         Aptitude::CmdLine::Ignore-Trust-Violations based upon the
         $apt_allow_unauthenticated option (Closes: #588035).  Thanks to
         Ansgar Burchardt for this patch.
       . Improve handling of alternative build dependencies and Provides
         using the interal build dependency resolver.  (Closes: #586275,
         #578810, #591187, #591188, #591189, #595863, #595866, #595872).
         Thanks to Ansgar Burchardt for this patch.
       . Remove virtual build-dependency resolving using the "internal"
         resolver using the static ALTERATIVES (%alternatives) list.  This
         was already limited, non-functional, and is obsoleted by the
         "aptitude" resolver.
       . Added the ability to resolve virtual dependencies using the
         "internal" build dependency resolver by calling "apt-cache
         --names-only" to get candidate packages to satisfy the
         dependency.  The first package (sorted alphabetically) is
         chosen (Closes: #595934).  This may be disabled using the
         $resolve_virtual option.
       . Use apt-get to remove build conflicts using the internal
         resolver and correctly reinstall post build.
     - Log the package source version as well as binary version in the
       package build log, since these may differ when making binNMUs
       (Closes: #569823).
     - Always attempt to install ESSENTIAL (build-essential) packages
       if they aren't already installed (Closes: #380113).
     - Don't end session on build failure (Closes: #567506).  This allows
       examination of build trees and cloned chroots on build failure,
       rather than unconditionally deleting them, which prevents this.
       Thanks to Jan-Marek Glogowski.
     - Remove support for parsing output of apt-get stderr
       (Closes: #571806).
     - Check for sbuild group membership using gid and supplementary
       groups (Closes: #599350).
   * buildd:
     - Read entire contents of pipes (Closes: #567821).  Thanks to
       Samuel Thibault.  This ensures receipt of complete log
       messages.
     - Replace/Breaks with ubuntu-dev-tools <= 0.83 (Closes: #590153).
       Thanks to Lorenzo De Liso.
   * buildd-vlog: Fix breakage (Closes: #567822).  Thanks to
     Samuel Thibault.  This fix introduces sleeps, and is not a
     complete or entirely optimal fix, but corrects the existing
     breakage.
   * Documentation:
     - Remove trailing 'q' from sbuild-createchroot(8) manual page
       (Closes: #579971).  Thanks to Salvatore Bonaccorso.
     - sbuild(1):
       . Remove and rewrite obsolete parts of the manpage
         (Closes: #599348).
       . .sbuildrc is no longer required for sbuild to function
         (Closes: #598054).
Checksums-Sha1: 
 1cee8af50caf25930606872c02016772430fc865 1265 sbuild_0.60.2-1.dsc
 7bde94207dc65085d95c096071dd971e0ba1c266 499416 sbuild_0.60.2.orig.tar.gz
 a0fddd5458378c1bf3c10dd2f5c060d1347741ed 20 sbuild_0.60.2-1.diff.gz
 06f8a42e99e2cfa35946cd1de3b9c89f6a490190 220524 libsbuild-perl_0.60.2-1_all.deb
 fcf694f80364cb73c4742da559e9bf490279deaa 224206 sbuild_0.60.2-1_all.deb
 ef142d6c8caca555130637013fccc1125b21f6c1 226112 wanna-build_0.60.2-1_all.deb
 c0791f78d117c4384b3b79effa996157ee5a5edc 217578 buildd_0.60.2-1_all.deb
Checksums-Sha256: 
 ff4c1c71a74a025931a5288b6646ac6c8e42cb320399409d3b19a9b79391440b 1265 sbuild_0.60.2-1.dsc
 cdaf4bbe5263be3edfc7c1705336ca7df2e3ebe42e18a80adb7f146ce2ca84fa 499416 sbuild_0.60.2.orig.tar.gz
 f61f27bd17de546264aa58f40f3aafaac7021e0ef69c17f6b1b4cd7664a037ec 20 sbuild_0.60.2-1.diff.gz
 77fb08c2c2b66c80bc4046a3b9f07685fbe66ce8411c51f68ef6a207c8fa64a8 220524 libsbuild-perl_0.60.2-1_all.deb
 2909b5421445d0c5466e7d36c4a3d3e7f09d6cc44bb4def827838882341f8312 224206 sbuild_0.60.2-1_all.deb
 84993adb3fc826f0a25af68b2cce0db069830482a2bd6aae0baf10a46c4ccceb 226112 wanna-build_0.60.2-1_all.deb
 4539f57bb2e842e16a5055fcfa722296783a0483733e8be166eba070b9d2d6dd 217578 buildd_0.60.2-1_all.deb
Files: 
 781ac6e9e147509747e69ae9205838bc 1265 devel extra sbuild_0.60.2-1.dsc
 e7b15af63ea04e9a96827b52debdd7e4 499416 devel extra sbuild_0.60.2.orig.tar.gz
 4a4dd3598707603b3f76a2378a4504aa 20 devel extra sbuild_0.60.2-1.diff.gz
 d3127a6a7d0120b51e1f8cdbae1bf946 220524 perl extra libsbuild-perl_0.60.2-1_all.deb
 069801e08b513bfb7bbe69508ef941ec 224206 devel extra sbuild_0.60.2-1_all.deb
 af05c50c0aefeb77f4440d73c163a4d4 226112 devel extra wanna-build_0.60.2-1_all.deb
 78484d82b9c582d3ecce2cdd7ab76210 217578 devel extra buildd_0.60.2-1_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEAREDAAYFAkzQJ/gACgkQVcFcaSW/uEjbXgCg9hu0dflgJBF9O2FWTsdJX80i
kmYAoITWqpZJ8q+xUJfVE3W7pZMhiB+z
=YubG
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.