NetBSD-Bugs archive

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

re: lib/54449: _pic.a libraries contain unwanted debug information



The following reply was made to PR lib/54449; it has been noted by GNATS.

From: Andreas Gustafsson <gson%gson.org@localhost>
To: matthew green <mrg%eterna.com.au@localhost>
Cc: gnats-bugs%netbsd.org@localhost, christos%NetBSD.org@localhost
Subject: re: lib/54449: _pic.a libraries contain unwanted debug information
Date: Sat, 10 Aug 2019 11:42:33 +0300

 matthew green wrote:
 > >  Try this: https://www.netbsd.org/~christos/strip-piclib.diff
 > 
 > why this?  if you're going to do something like this then actually
 > save the data into the debug sets.  anyone who is using these is
 > likely happy they have debug info, so removing it is likely to be
 > sad-making.
 
 When I look in /usr/libdata/debug/usr/lib on my 8.1 system, I only
 see .debug files for the .so libraries, so it looks like we are not
 shipping debug information for .a nor _p.a.  I think we should be
 consistent and either ship debug information for all static libraries,
 or for none of them.  Shipping it for .so and _pic.a but not .a nor
 _p.a would just be weird.
 
 > but i really think the saner move is to leave this as-is and
 > change the default for MKPICINSTALL to "no".  the vast majority
 > of people are never going to use them so it's wasteful to install,
 > debug info or not.
 
 I don't have an opinion on whether or not that should be done, but it
 does not change the need for Christos' patch - even if we set the
 default of MKPICINSTALL to "no", the present bug will still affect
 those who override the default to "yes", so it still needs to be fixed.
 
 So, to keep this PR focused on the specific issue in its subject, if
 you want those other changes, please open your own PRs for "static
 libraries should have debug info in /usr/libdata/debug" and/or
 "MKPICINSTALL should default to no".
 -- 
 Andreas Gustafsson, gson%gson.org@localhost
 


Home | Main Index | Thread Index | Old Index