[buildd-tools-devel] sbuild: multiarch build support

[Geoffrey Thomas]

On Fri, 29 Aug 2014, Wookey wrote:

> The patch simply identifies explicit arch-qualified build-deps (of the
> form libfoo:arch) and if present runs dpkg --add-foreign-architecture
> arch (as it already does for cross-builds). Apt takes care of the rest.
>
> The second part (for removing the arch after the build (on permanent
> chroots)) needs more work as usually there are binaries of the foreign
> arch installed so the arch-removal fails.
>
> Comments on the code are welcome.

Maybe these should be obvious, but I'm curious about these two things I 
don't understand:

1. As far as I can tell, while deps_parse returns a nested data structure, 
it can never return a _recursive_ data structure. It's parsing a single 
Build-Depends line, and not looking for dependencies of any build-depends, 
right? So you do get a data structure with AND and OR nodes, but nothing 
that should be more than two levels nested. Are you actually seeing 
recursion in practice?

2. While we probably don't want to go this route, how does this differ 
from simply adding all architectures in the archive as foreign 
architectures, unconditionally? Is there a risk of builds pulling in 
cross-arch dependencies when not needed?

-- 
Geoffrey Thomas
https://ldpreload.com
geofft at ldpreload.com