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

Bug#69487: the example for using nostrip in DEB_BUILD_OPTIONS is incorrect



On Wed, Aug 23, 2000 at 02:59:39PM -0500, Steve Greenland wrote:
> On 20-Aug-00, 15:24 (CDT), Ben Collins <bcollins@debian.org> wrote: 
> > The nostrip check needs to be inside the debug check. Because of you are
> > not compiling with debugging turned on, there's no reason to not strip the
> > binaries. So (note, the blank should go first):
> > 
> > ifneq "" "$(findstring debug,$(DEB_BUILD_OPTIONS))"
> >   CFLAGS += -g
> >   ifeq "" "$(findstring nostrip,$(DEB_BUILD_OPTIONS))"
> >     INSTALL += -s
> >   endif
> > endif
> 
> While I agree with the philosophy, this code snippet is wrong, as
> it will add the "-s" iff DEB_BUILD_OPTIONS includes "debug" but not
> "nostrip".
> 
This makes perfect sense, as when you strip a binary, you kill the
symbol table and hence -g is useless

I could be wrong with current systems, but as I remember it, this is 
the correct method


Frank aka Myth

Attachment: pgpLCf3xyuBn_.pgp
Description: PGP signature


Reply to: