On Tue, Dec 28, 2010 at 02:29:43PM +0100, Ulrich Sp??rlein wrote:
> On Sat, 25.12.2010 at 08:42:38 +0000, Konstantin Belousov wrote:
> > Author: kib
> > Date: Sat Dec 25 08:42:38 2010
> > New Revision: 216694
> > URL: http://svn.freebsd.org/changeset/base/216694
> > 
> > Log:
> >   Add a hook to pass debug flags to the build of rtld when doing make in
> >   the rtld directory.
> >   
> >   Reviewed by:      kan
> 
> Please revert this, $(VAR) is against style, and passing DEBUG_FLAGS is
> the canonical way to achieve what you wanted, eg. make DEBUG_FLAGS=-g is
> working just fine for me.
DEBUG_FLAGS=-g might be fine for you, but it does not do what rtld debugging
needs. Look at the -DDEBUG.

If you want ${} instead of $(), feel free to change.

> 
> > Modified:
> >   head/libexec/rtld-elf/Makefile
> > 
> > Modified: head/libexec/rtld-elf/Makefile
> > ==============================================================================
> > --- head/libexec/rtld-elf/Makefile  Fri Dec 24 21:31:18 2010        
> > (r216693)
> > +++ head/libexec/rtld-elf/Makefile  Sat Dec 25 08:42:38 2010        
> > (r216694)
> > @@ -34,7 +34,7 @@ CFLAGS+=  -fPIC
> >  .else
> >  CFLAGS+=   -fpic
> >  .endif
> > -CFLAGS+=   -DPIC
> > +CFLAGS+=   -DPIC $(DEBUG)
> >  LDFLAGS+=  -shared -Wl,-Bsymbolic
> >  DPADD=             ${LIBC_PIC}
> >  LDADD=             -lc_pic -lssp_nonshared

Attachment: pgplkXmBVZsuh.pgp
Description: PGP signature

Reply via email to