On Sat, Aug 21, 2010 at 01:48:05PM +0000, Rui Paulo wrote: > Author: rpaulo > Date: Sat Aug 21 13:48:04 2010 > New Revision: 211561 > URL: http://svn.freebsd.org/changeset/base/211561 > > Log: > Link drti.o with a PIC version of libelf. This is needed because > drti.o depends on libelf and this avoids linking every other drti.o > program (namely programs with USDT probes) with libelf. What is wrong with having libelf as dependency of the resulting binary ?
>
> Sponsored by: The FreeBSD Foundation
>
> Modified:
> head/cddl/lib/drti/Makefile
>
> Modified: head/cddl/lib/drti/Makefile
> ==============================================================================
> --- head/cddl/lib/drti/Makefile Sat Aug 21 13:42:12 2010
> (r211560)
> +++ head/cddl/lib/drti/Makefile Sat Aug 21 13:48:04 2010
> (r211561)
> @@ -8,13 +8,20 @@ FILESOWN= ${LIBOWN}
> FILESGRP= ${LIBGRP}
> FILESMODE= ${LIBMODE}
> FILESDIR= ${LIBDIR}/dtrace
> -CLEANFILES= ${FILES}
> +CLEANFILES= ${FILES} ${FILES}.tmp
>
> CFLAGS+= -I${.CURDIR}/../../../sys/cddl/compat/opensolaris \
> -I${.CURDIR}/../../../cddl/compat/opensolaris/include \
> -I${OPENSOLARIS_USR_DISTDIR}/head \
> -I${OPENSOLARIS_USR_DISTDIR}/lib/libctf/common \
> -I${OPENSOLARIS_USR_DISTDIR}/lib/libdtrace/common \
> - -I${OPENSOLARIS_SYS_DISTDIR}/uts/common
> + -I${OPENSOLARIS_SYS_DISTDIR}/uts/common \
> + -DPIC -fpic
> +
> +.c.o:
> + ${CC} ${CFLAGS} -c ${.IMPSRC}
> + mv ${FILES} ${FILES}.tmp
> + ${LD} -o ${FILES} -r ${FILES}.tmp ${LIBELF_PIC}
> + rm -f ${FILES}.tmp
>
> .include <bsd.prog.mk>
pgphXSrLfYTf0.pgp
Description: PGP signature
