[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#752897:



> On Tue, Aug 26, 2014 at 12:30 PM, Tobias Frost <tobi@debian.org> wrote:
>> Hi,
>>
>> Well, appearantly malat uploaded in the meantime...
>>
>> Malat, this is great, but could you also drop a note to the BTS if you
>> are doing that to avoid double work, especially if there is activiy and
>> the owner of the RFS-bug has been set to indicate that someone is
>> working on it.. Thanks ;).
>>
>> Regarding the package, unfortunately, the way Build-Depends-Indep is
>> used will likely not work: You need also to work on d/rules.
>> I'm not sure how the buildds handle Build-Depends-Indep, but if they
>> don't install them the package will FTBFS.
>> (Try dpkg-buildpackage -B with doxygen not installed.)
>>
>> Most likely d/rules need to detect if it is supposed to build the docs;
>> You can use dh_listpackages or, an easy way, detect doxygen and act
>> accordingls: For example something like that: (note, I did not test that
>> below)
>>
>> override_dh_auto_configure:
>>         if [ -x /usr/bin/doxygen ] ; then dh_auto_configure --
>> -DENABLE_DOCS:BOOL=ON -DENABLE_STANDARD_ALLOCATOR:BOOL=ON ;\
>>           else dh_auto_configure -- -DENABLE_DOCS:BOOL=OFF
>> -DENABLE_STANDARD_ALLOCATOR:BOOL=ON; fi
>>
>> override_dh_auto_build:
>>         if [ -x /usr/bin/doxygen ] ; then dh_auto_build -- doc ; else
>> dh_auto_build ; fi
>
> This is unnecessary; just use an override_dh_auto_build-indep target
> in debian/rules. Assuming you have doxygen listed in
> Build-Depends-Indep, you do not have to manually check for doxygen.
>
> Regards,
> Vincent
>

Thanks Vincent for the info; Learned another thing :)

However, in this case the package needs also different cmake options at
dh_auto_configure-time, so only override_dh_auto_build-indep will not work as
you need to disable DENABLE_DOCS during cmake configuration.
I just wondering if there is also a dh_auto_configure-indep target... (I cant
test because I'm not near my PC right now)

--
tobi


Reply to: