[SCM] mplayer packaging branch, master, updated. debian/1.0.rc2+svn20090303-2-15-gfa424e6

[Felipe Sateler]

El 23/03/09 21:19 Reinhard Tartler escribió:
> Felipe Sateler <fsateler at gmail.com> writes:
> > El 23/03/09 20:37 Reinhard Tartler escribió:
> >> mennucc1 at users.alioth.debian.org writes:
> >> > The following commit has been merged in the master branch:
> >> > commit fa424e6b54409bc27cb4ad3631d40abf6fcaa154
> >> > Author: Andrea Mennucc <mennucc1 at debian.org>
> >> > Date:   Mon Mar 23 10:06:03 2009 +0100
> >> >
> >> >     move documentation-related dependencies into Build-Depends ,
> >> >     otherwise ./configure fails
> >>
> >> the rationale isn't quite correct. It's not configure that fails, but
> >> our debian/rules file insists on building the documentation even on the
> >> buildds. Which is plain wrong and a bug *we* have to fix.
> >
> > Policy and dpkg are inconsistent in this sense. dpkg-buildpackage
> > actually calls debian/rules build instead of debian/rules build-arch when
> > called with -B. So Build-Depends-Indep is not really useful right now
> > (unless you do ugly tricks like building documentation in the
> > binary-indep target).
>
> how would that work?
>
> could you perhaps post a patch demonstrating that ugly trick?

I haven't tested this (I don't have the build-deps installed and my link is 
awful right now), but the attached patch should do the trick. The thing is as 
follows: packages are built in two rounds: first the package is compiled, and 
then the binary packages are constructed. This is because packages should not 
be compiled as root, but you need (fake)root priviledges to install and do 
other stuff. When calling dpkg-buildpackage with the -B option (which is what 
the buildds do), dpkg will not call "debian/rules binary", but "debian/rules 
binary-indep". However, since policy does not mandate the existence of 
build-{arch,indep} dpkg-buildpackage just calls "debian/rules build". This 
renders B-D-I rather useless. See bug#229357 for more discussion on the 
issue. The solution works because install-indep is not called by build-indep, 
but rather binary-indep, which won't get called in the buildds. The downside 
is that the documentation will be built as root in the developers machine.

Saludos,
Felipe Sateler
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch
Type: text/x-diff
Size: 649 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20090323/b7729041/attachment.diff 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20090323/b7729041/attachment.pgp