Your message dated Wed, 24 Aug 2016 11:50:31 +0000
with message-id <[email protected]>
and subject line Bug#834497: fixed in sbuild 0.71.0-1
has caused the Debian Bug report #834497,
regarding sbuild: apt, aspcud, xapt resolvers dropping extra repositories
before using them
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
834497: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=834497
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: sbuild
Version: 0.70.0-1
Severity: important
Tags: patch
Hi,
The current version of sbuild is cleaning up the extra repositories before
using them. This was introduced by 0e71b40, adding extra repositories unlink
to cleanup_apt_archive, which is called at the end of the resolvers
install_deps', which in turn is called by install_core_deps and
install_main_deps.
Since a normal build has a install_core_deps call before the
install_main_deps, the extra repositories are missing for the main part.
The aptitude resolver is not affected as it's not calling the
cleanup_apt_archive (I'm assuming that this relays on Build calling cleanup at
the right moments), the attached patch removes the cleanup_apt_archive
invocation from the resolvers assuming that the aptitude approach is correct.
Happy hacking,
-- System Information:
Debian Release: stretch/sid
APT prefers unstable-debug
APT policy: (500, 'unstable-debug'), (500, 'buildd-unstable'), (500,
'testing'), (500, 'stable'), (50, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386, armhf
Kernel: Linux 4.6.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Init: systemd (via /run/systemd/system)
Versions of packages sbuild depends on:
ii adduser 3.115
ii apt-utils 1.3~pre2
ii libsbuild-perl 0.70.0-1
pn perl:any <none>
Versions of packages sbuild recommends:
ii debootstrap 1.0.81
ii fakeroot 1.21-1
Versions of packages sbuild suggests:
ii deborphan 1.7.28.8-0.3
ii wget 1.18-2
-- Configuration Files:
/etc/sbuild/sbuild.conf changed [not included]
-- no debconf information
>From e9789f49aaf6828c817976c4982c29c72e8c2765 Mon Sep 17 00:00:00 2001
From: Maximiliano Curia <[email protected]>
Date: Tue, 16 Aug 2016 11:48:09 +0200
Subject: [PATCH] Avoid dropping extra repositories before using them
apt aspcud and xapt resolvers were dropping the extra repositories since
0e71b40 was introduced, the aptitude resolver is not affected since it's
not calling the cleanup_apt_archive, assuming this is the correct
approach, this patch removes the cleanup_apt_archive in the resolvers.
---
lib/Sbuild/AptResolver.pm | 2 --
lib/Sbuild/AspcudResolver.pm | 2 --
lib/Sbuild/XaptResolver.pm | 2 --
3 files changed, 6 deletions(-)
diff --git a/lib/Sbuild/AptResolver.pm b/lib/Sbuild/AptResolver.pm
index b10dca7..a362194 100644
--- a/lib/Sbuild/AptResolver.pm
+++ b/lib/Sbuild/AptResolver.pm
@@ -104,8 +104,6 @@ sub install_deps {
}
}
- $self->cleanup_apt_archive();
-
return $status;
}
diff --git a/lib/Sbuild/AspcudResolver.pm b/lib/Sbuild/AspcudResolver.pm
index c846a9c..6d51f40 100644
--- a/lib/Sbuild/AspcudResolver.pm
+++ b/lib/Sbuild/AspcudResolver.pm
@@ -117,8 +117,6 @@ sub install_deps {
}
cleanup:
- $self->cleanup_apt_archive();
-
return $status;
}
diff --git a/lib/Sbuild/XaptResolver.pm b/lib/Sbuild/XaptResolver.pm
index 139c202..09d866f 100644
--- a/lib/Sbuild/XaptResolver.pm
+++ b/lib/Sbuild/XaptResolver.pm
@@ -94,8 +94,6 @@ sub install_deps {
}
}
- $self->cleanup_apt_archive();
-
return $status;
}
--
2.8.1
--- End Message ---
--- Begin Message ---
Source: sbuild
Source-Version: 0.71.0-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.
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 [email protected],
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Johannes Schauer <[email protected]> (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 [email protected])
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256
Format: 1.8
Date: Wed, 24 Aug 2016 13:42:26 +0200
Source: sbuild
Binary: libsbuild-perl sbuild buildd
Architecture: source
Version: 0.71.0-1
Distribution: unstable
Urgency: medium
Maintainer: Debian buildd-tools Developers
<[email protected]>
Changed-By: Johannes Schauer <[email protected]>
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
Closes: 827315 833285 833286 833390 833547 833549 833552 834330 834497 834515
834704 834898
Changes:
sbuild (0.71.0-1) unstable; urgency=medium
.
* add autopkgtest to Suggests
* Undo exporting private keys in armored ASCII format as introduced by the
last release. It turns out, that this is not necessary to support gpg
cross-version compatibility. This should fix problems with gpg 2.x as well
as chroots not having gpg installed as apt stopped depending on it.
(closes: #827315, #834898)
* Undo the SUITE-VARIANT syntax of sbuild-createchroot as introduced by the
last release. It turns out that overloading the suite name argument with a
variant name was a bad idea because there exist distributions with a minus
in its name. Variant names can now be specified using the --chroot-prefix
option which works analogous to the --chroot-suffix option. (closes:
#834515)
* Add percent escape support to piuparts, autopkgtest and adt-virt options.
(closes: #833285, #833286)
* Add sbuild-destroychroot.
* Allow running autopkgtest when sbuild downloads the .dsc (closes: #833549)
* Run apt-cache without --only-source (closes: #834704)
* Avoid dropping extra repositories before using them (closes: #834497)
* lib/Sbuild/ResolverBase.pm: Fix copypaste error s/public/private/ (closes:
#834330)
* lib/Sbuild/ChrootSetup.pm: don't prompt the user during --keygen (closes:
#833547)
* Add an option to not prefix piuparts and autopkgtest with anything at all
(closes: #833390)
* lib/Sbuild/Build.pm: add red/green coloring to log when piuparts or
autopkgtest succeed/fail (closes: #833552)
Checksums-Sha1:
036f44a4263a0ffe0cd0a3b417885e6c2e49aa85 2359 sbuild_0.71.0-1.dsc
3fd94b836f2726199cbb8fbb54e3d5ba91f88b3e 635520 sbuild_0.71.0.orig.tar.gz
f9272abf45192982e1f9b27bd0644129f7e06733 54760 sbuild_0.71.0-1.debian.tar.xz
Checksums-Sha256:
6636eec05e39646abca30e33012339630efd021ae00dd696b562f1816b8f00ec 2359
sbuild_0.71.0-1.dsc
7001e7ecb731be012757b1be96813249448b944fe820dcd14d4fd68db05dd874 635520
sbuild_0.71.0.orig.tar.gz
a8982f193277792d0bdae0287212cf5c88ed71152a907d0e647323e1c3f9a1b1 54760
sbuild_0.71.0-1.debian.tar.xz
Files:
a64281c4d8e1c9ee606997f305126289 2359 devel extra sbuild_0.71.0-1.dsc
0ba688ac2aea3f64509a94e6c7119476 635520 devel extra sbuild_0.71.0.orig.tar.gz
9ae3426cf9c5d0bc3e513a15a83f8ae6 54760 devel extra
sbuild_0.71.0-1.debian.tar.xz
-----BEGIN PGP SIGNATURE-----
iQIuBAEBCAAYBQJXvYh/ERxqb3NjaEBkZWJpYW4ub3JnAAoJEPLLpcePvYPhrCsP
/3DvnC67gc8YbfKb5K+1SMS/uaUJgd0JjDy2qykJTC3iFc+c8DyLFNXXl8BkEa7u
gUUh4hQ8omqnOwPvjl4/7U/cDcqVpjvgE3MsgJxytvDB945wG1TjbMuWZkx1d3tj
SkWlvnaWbWusVyzUrYELVZpMCbcxESKubO/IHixoNdQG1CkgxkoZmtuuEgMMSJx+
0QTNEjgBSbECXbXdGygTIaM/cXTyXcN7GY2dfDgtuHN+Q8vO1cAYKPAmeLU2ViEb
XxeuU/Uxlqdx/sC/Tlpz8Z+TEw//lW7Wq/UvsyUT2UPXjrqSwi+4Da11f1YwoH1p
kisCQkmi9ZP2octR5XOLRI2iAVE9O81U1Vdk7yPEyWuyOBQhTlXrVNpwQmrskT4F
/D5eoGUMHtvWiIwzr9OZgDufEbBMDnN1/kRYvdJ1bo3Q6Mu0JdkW82aE/RZYo5ng
6f4CUgAx/7cDLehCPTm8mX3CVANl47Ox8uoL8lSyc2eVejf3MBu/3o8DF3aHDyJr
pIN417DT/c+Hfvi7/maGt30ty+1h7UYD8x9XzctmdEeYoEHg4iiEfl7i7HHSYc+V
vdiAefiMHVtfUtYeM2rsJjz8mS7RtnfPZkwxB3hFU3ud08dodpA7T1Wm4sBkvHCH
OnFtFMBtuX8rR9KKjUcPB1P0LlxX1p3lh/pNV4CV0php
=4NRL
-----END PGP SIGNATURE-----
--- End Message ---
