Steven Kreuzer <skreu...@freebsd.org> writes:

> With the help of jhb@, I have a port of gdb 7.1 that has been modified
> to recognize freebsd threads. I would like to get this added to the
> ports tree but I would like some feedback as to how it should be
> added.
>
> Currently, the latest version of gdb in the ports tree is 6.6 and its
> under devel/gdb66. I was thinking of adding this as gdb71 but the
> latest version of the 7.x branch is 7.2 and someone already created a
> port for it (ports/151077) but it doesn't recognize threads.
>
> So, for the 7 branch, would it make sense to check in version 7.1 as
> devel/gdb7 and eventually update it to 7.2, or should it be checked in
> as devel/gdb71?
>
> shar file can be found at http://exit2shell.com/~skreuzer/patches/gdb71.shar

- fix build with devel/readline[*]
- explicitly disable xml/unwind/python support
- drop gettext dep, unused

Note, lang/gcc* (and lang/clang) searches in LOCALBASE by default unlike
gcc in base. Having it in *FLAGS by default reduces difference between
build using ports compiler and base one.

[*] 
http://docs.freebsd.org/cgi/mid.cgi?AANLkTinkwsFXpXE=CxcDjTGPVo3aa8EQ1_ve0h3NiB5E

%%
--- gdb71/Makefile
+++ gdb71/Makefile
@@ -17,15 +17,21 @@ COMMENT=    GNU GDB of newer version than comes with the 
system
 
 USE_BZIP2=     yes
 USE_GMAKE=     yes
-USE_GETTEXT=   yes
 USE_ICONV=     yes
 GNU_CONFIGURE= yes
-CONFIGURE_ENV+=        CONFIGURED_M4=m4 CONFIGURED_BISON=byacc
+CONFIGURE_ENV+=        CONFIGURED_M4=m4 CONFIGURED_BISON=byacc 
LDFLAGS="${LDFLAGS}"
 CONFIGURE_ARGS=        --program-suffix=${PORTVERSION:S/.//g} \
+               --with-libiconv-prefix=${LOCALBASE} \
+               --with-system-readline \
+               --without-expat \
+               --without-libunwind \
+               --without-python \
                --enable-target=all \
-               --enable-tui --with-libiconv-prefix=${LOCALBASE}
+               --enable-tui
 CFLAGS:=       ${CFLAGS:C/ +$//}       # blanks at EOL creep in sometimes
+CFLAGS+=       -I${LOCALBASE}/include
 CFLAGS+=       -DRL_NO_COMPAT
+LDFLAGS+=      -L${LOCALBASE}/lib
 EXCLUDE=       dejagnu expect readline sim texinfo intl
 EXTRACT_AFTER_ARGS=| ${TAR} -xf - ${EXCLUDE:S/^/--exclude /}
 VER=   ${PORTVERSION:S/.//}
@@ -36,6 +42,11 @@ ONLY_FOR_ARCHS=      i386 amd64      # untested elsewhere, 
might work
 
 .include <bsd.port.pre.mk>
 
+# XXX: add OSVERSION check after readline is removed from base
+.if exists(${LOCALBASE}/lib/libreadline.so)
+LIB_DEPENDS+=  readline.6:${PORTSDIR}/devel/readline
+.endif
+
 .if ${ARCH} == "amd64"
 CONFIGURE_TARGET=      x86_64-portbld-freebsd${OSREL}
 .endif
@@ -43,11 +54,6 @@ CONFIGURE_TARGET=    x86_64-portbld-freebsd${OSREL}
 post-patch:
        @${REINPLACE_CMD} -e 's/$$/ [GDB v${PORTVERSION} for FreeBSD]/' \
                ${WRKSRC}/gdb/version.in
-       @${REINPLACE_CMD} -e 's/^READLINE/#READLINE/' \
-               -e 's/^readline/#readline/' \
-               -e 's,^LIBICONV =.*,LIBICONV=-L${LOCALBASE}/lib -liconv,' \
-               ${WRKSRC}/gdb/Makefile.in
-       ${ECHO} 'READLINE = -lreadline' >> ${WRKSRC}/gdb/Makefile.in
        @${CP} ${FILESDIR}/fbsd-threads.c ${WRKSRC}/gdb
 
 do-install:
%%
_______________________________________________
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Reply via email to