[buildd-tools-devel] Bug#714149: sbuild: Let users push additional s to the build dependencies dummy archivearchive
Geoffrey Thomas
gthomas at mokafive.com
Tue Jul 2 06:30:37 UTC 2013
On Wed, 26 Jun 2013, Emanuele Aina wrote:
> When building related set of packages most people resort to set up a local
> archive to enable the newly built packages to depend on locally built
> dependencies for which the needed version has not hit the main archive yet.
>
> Since sbuild already sets up a dummy archive for build dependencies, it could
> also allow users to specify a list of packages that should be pushed there.
>
> To do so I've added an --add-extra-package option that accepts a debfile and
> can be used multiple times. It already allowed me to locally build packages
> which depend on stuff not yet in the archive without any additional setup.
>
> The git repo (`add-extra-package` branch) with the patches is at
> http://cgit.collabora.com/git/user/em/sbuild/log/?h=add-extra-package
>
> I also took the chance to refactor a bit the dummy archive handling and split
> it out of the ResolverBase class. Even if not strictly needed it allowed me to
> have a clear idea of the involved pieces and I think that the result is a bit
> easier to understand and more flexible.
>
> Comments and suggestions welcome, I'm not very proficient in Perl. :)
Hi Emanuele,
This looks like exactly what I was going for with this much smaller
patch, written for bug #700522:
https://github.com/geofft/sbuild/commit/3f7ecfde5e07c66ec1d7e88fac7423f94400f68b
See http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700522
Want to merge these two wishlist bugs? I haven't looked in detail at your
refactoring, but it felt like my patch was adding this in a slightly
questionable manner, so I'd be happy for yours to get merged instead if
it's preferable.
Also want to take a crack at --add-repository? :-) My plan there was to
patch `apt-get update` to let you update a single repository, clean up the
existing messy code that you refactored into force_update_archive_list()
to use that interface, and then allow adding a sources.list.d file and
updating that. Unfortunately the most obvious way to implement that apt
patch ends up with apt forgetting about every other source, and I didn't
have time to try harder.
--
Geoffrey Thomas
gthomas at mokafive.com
More information about the Buildd-tools-devel
mailing list