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
build-indep-stamp:
+ifneq (,$(findstring libcsound64-doc,$(shell dh_listpackages)))
doxygen
+endif
touch $@
# Put aside upstream cruft before build but after licensecheck
TBH, this solution seems acceptable to me.
--
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
