On Mon, Aug 02, 2010 at 19:23:59 (EDT), Felipe Sateler wrote:

> On 02/08/10 18:26, Adam D. Barratt wrote:
>> On Sun, August 1, 2010 12:33, Felipe Sateler wrote:
>>> Copying this to the appropriate bug...
>> [...]
>>> Indeed. How do you suggest working through this? Facts:
>>> 1. The build hangs unpredictably on a doxygen call.
>>> 2. The doxygen call is in build-indep (so it is not strictly necessary
>>> for binary only builds, but gets executed anyway).
>>> I can move the doxygen call away from there into binary-indep, but that
>>> feels like a hack to me.
>> Does the documentation actually differ across architectures?
> No, it doesn't.
>>  If the doxygen calls are simply for generating the contents of
>> libcsound64-doc then arranging for them to only occur when the
>> binary-indep packages are being built sounds like a sane solution; if
>> I'm missing something obvious, then someone please apply the relevant
>> cluebat. :-)
> Well, it feels hacky to build stuff in the binary stage. Also, it seems
> to me that what we are doing is re-hiding a bug that is somewhere else.
> If we really want to do that, please tell me and I'll prepare a new
> upload with that.

Mehdi proposed this approach:

diff --git a/debian/rules b/debian/rules
index 726227e..626b8d6 100755
--- a/debian/rules
+++ b/debian/rules
@@ -163,7 +163,9 @@ DEB_SCONS_INSTALL_TARGET = install=1
 build-indep: build-indep-stamp
+ifneq (,$(findstring libcsound64-doc,$(shell dh_listpackages)))
        touch $@
 # Put aside upstream cruft before build but after licensecheck

TBH, this solution seems acceptable to me.

Reinhard Tartler, KeyID 945348A4

pkg-multimedia-maintainers mailing list

Reply via email to