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 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
