[Debian-med-packaging] Bug#640667: excessive build time -- not splitting the generated swig files and building with -O3

[Matthias Klose]

On 09/08/2011 03:59 AM, Steve M. Robbins wrote:
> Hi,
> 
> Thanks for the patch!
> 
> On Wed, Sep 07, 2011 at 07:14:53PM +0200, Matthias Klose wrote:
> 
>> patch at 
>> http://launchpadlibrarian.net/79253000/insighttoolkit_3.20.0-13_3.20.0-13ubuntu1.diff.gz
>
>> 
> These look like valuable improvements.  I have some questions:
> 
> 1. Why not build as RelWithDebInfo for all architectures?

didn't want to touch it.  Just building without this flag results in some
subprojects being built *without* optimization. didn't investigate further,
and don't plan to.

> 2. What does this have to do with LP: #641126?

there's still a linker bug on armel, there are duplicates/other reports.

> 3. Does this fix this bug report of excessive build time?  Is using -O2
> enough to fix this?

it looks like it keeps the cc1plus processes below 1GB RAM, so yes.

>> note that the ix86 builds are still done with -NDEBUG and with -g, which
>> seems to be a policy violation.
> 
> I couldn't find NDEBUG in Policy.  What clause is violated?

it's the missing -g, policy 10.1