[Pkg-corba-devel] -dbg libraries

[Thomas Girard]

Hi Floris,

On Thu, Oct 11, 2007 at 10:22:25PM +0100, Floris Bruynooghe wrote:
> > The attached patch (uncommitted) adds `-g' using CC and CXX configure
> > variables. If you're fine with it I'll commit it and create missing
> > -dbg packages.
> 
> That is a neat solution.  Although on its own it will just create add
> debugging sybols to the normal libs which is not what we want I think.
> 
> Yesterday (before I read this) I also worked on this on the train,
> creating separate -dbg pacakges (and discovering a few files to be
> cleaned up in the mean time too) for the library packages.  I've
> commited that work to a separate branch so you can have a look at it
> too, altough it is not finished yet.  It compiles but there are
> lintian errors and I haven't actually checked the binary packages at
> all yet.  But since I'm away over the weekend I thought it was better
> to commit it to a branch, I'll then try to merge the required changed
> atomically(ish) (with more verbose logs and a changelog entry ;-)) to
> the trunk later if we like it.
> 
> Anyway, if you think that looks useful (I think so) would you want to
> commit/merge your diff to that branch?  I think I prefer your way of
> adding flags then my usage of sed, and the other changes in your diff
> seem good too.  I think having second target
> "build-dbg/config.status:" that would use your trick to make CC="gcc
> -g" would be the cleanest, so that configure gets called twice.
> 
> The only difference is that my sed trick compiles with just -g while
> your trick ends up using -g -O2.  But I think that's fine.

I will be away for the week-end, but I had a quick look at your changes:
it seems your changes are compiling libraries twice. Recent versions of
gdb are able to handle debugging symbols put in /usr/lib/debug, and we
should use dh_strip(1) to extract those debugging symbols. (That's what
we do in stlport5.1 for instance.)

Besides, according to [1], programs should be compiled with `-g -O2',
then stripped (or in our case, move the debugging symbols into -dbg
packages).

I'll investigate further next week.

Cheers,

Thomas

[1] http://www.debian.org/doc/debian-policy/ch-files.html#s10.1