From nobody@FreeBSD.org  Wed Jun 20 12:28:35 2012
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 4D7B9106564A
	for <freebsd-gnats-submit@FreeBSD.org>; Wed, 20 Jun 2012 12:28:35 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from red.freebsd.org (red.freebsd.org [IPv6:2001:4f8:fff6::22])
	by mx1.freebsd.org (Postfix) with ESMTP id 2ED508FC19
	for <freebsd-gnats-submit@FreeBSD.org>; Wed, 20 Jun 2012 12:28:35 +0000 (UTC)
Received: from red.freebsd.org (localhost [127.0.0.1])
	by red.freebsd.org (8.14.4/8.14.4) with ESMTP id q5KCSZ2i048071
	for <freebsd-gnats-submit@FreeBSD.org>; Wed, 20 Jun 2012 12:28:35 GMT
	(envelope-from nobody@red.freebsd.org)
Received: (from nobody@localhost)
	by red.freebsd.org (8.14.4/8.14.4/Submit) id q5KCSYc8048070;
	Wed, 20 Jun 2012 12:28:34 GMT
	(envelope-from nobody)
Message-Id: <201206201228.q5KCSYc8048070@red.freebsd.org>
Date: Wed, 20 Jun 2012 12:28:34 GMT
From: Luca Pizzamiglio <luca.pizzamiglio@gmail.com>
To: freebsd-gnats-submit@FreeBSD.org
Cc: bgmoser@codexterous.com
Subject: [MAINTAINER] OptionsNG and fix ports/168444
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         169265
>Category:       ports
>Synopsis:       [MAINTAINER] devel/gdb: OptionsNG and fix ports/168444
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    scheidell
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Jun 20 12:30:11 UTC 2012
>Closed-Date:    Sat Jun 23 10:00:15 UTC 2012
>Last-Modified:  Sat Jun 23 10:00:15 UTC 2012
>Originator:     Luca Pizzamiglio
>Release:        FreeBSD 8.3-STABLE
>Organization:
>Environment:
>Description:
Adopting the OptionsNG framework
Fix ports/168444 that could be closed
Bumping PORTREVISION.
>How-To-Repeat:

>Fix:
Adopt the patch

Patch attached with submission follows:

diff -rNu devel/gdb.orig/Makefile devel/gdb/Makefile
--- devel/gdb.orig/Makefile	2012-05-01 17:53:46.000000000 +0200
+++ devel/gdb/Makefile	2012-06-20 14:26:15.000000000 +0200
@@ -8,6 +8,7 @@
 
 PORTNAME=	gdb
 PORTVERSION=	7.4.1
+PORTREVISION=	1
 CATEGORIES=	devel
 MASTER_SITES=	${MASTER_SITE_GNU:S,$,:gdb,}
 MASTER_SITE_SUBDIR=gdb/:gdb
@@ -45,38 +46,36 @@
 CFLAGS+=	-isystem ${LOCALBASE}/include
 LDFLAGS+=	-L${LOCALBASE}/lib
 
-OPTIONS=	DEBUG "Build with debugging symbols" off \
-		EXPAT "Enable XML parsing for metadata" off \
-		PYTHON "Enable Python support" off \
-		THREAD "Enable Thread support" on \
-		GDB_LINK "Create the gdb link" on
+OPTIONS_DEFINE=	DEBUG EXPAT PYTHON THREADS GDB_LINK
+GDB_LINK_DESC=	Create the gdb link
+OPTIONS_DEFAULT=	THREADS GDB_LINK
 
 .include <bsd.port.options.mk>
 
-.if defined(WITH_GDB_LINK)
+.if ${PORT_OPTIONS:MGDB_LINK)
 PLIST_SUB+=	GDB_LINK=""
 .else
 PLIST_SUB+=	GDB_LINK="@comment "
 .endif
 
-.if defined(WITH_PYTHON)
+.if ${PORT_OPTIONS:MPYTHON}
 USE_PYTHON=	2.5-2.7
 .endif
 
 .include <bsd.port.pre.mk>
 
-.if defined(WITH_DEBUG)
+.if ${PORT_OPTIONS:MDEBUG}
 CFLAGS+=	-g
 .endif
 
-.if defined(WITH_EXPAT)
+.if ${PORT_OPTIONS:MEXPAT}
 LIB_DEPENDS+=	expat.6:${PORTSDIR}/textproc/expat2
 CONFIGURE_ARGS+=	--with-expat=yes
 .else
 CONFIGURE_ARGS+=	--without-expat
 .endif
 
-.if defined(WITH_PYTHON)
+.if ${PORT_OPTIONS:MPYTHON}
 CONFIGURE_ARGS+=	--with-python=${PYTHON_CMD}
 PLIST_SUB+=		PYTHON=""
 .else
@@ -91,7 +90,7 @@
 post-patch:
 	@${REINPLACE_CMD} -e 's/$$/ [GDB v${PORTVERSION} for FreeBSD]/' \
 		${WRKSRC}/gdb/version.in
-.if !defined(WITHOUT_THREAD)
+.if empty(PORT_OPTIONS:MTHREADS)
 	@${CP} ${FILESDIR}/fbsd-threads.c ${WRKSRC}/gdb/
 .endif
 	@${CP} ${FILESDIR}/amd64bsd-nat.h ${WRKSRC}/gdb
@@ -100,10 +99,12 @@
 	${INSTALL_PROGRAM} ${WRKSRC}/gdb/gdb ${PREFIX}/bin/gdb${VER}
 	${LN} ${PREFIX}/bin/gdb${VER} ${PREFIX}/bin/gdbtui${VER}
 	${INSTALL_MAN} ${WRKSRC}/gdb/gdb.1 ${MAN1PREFIX}/man/man1/gdb${VER}.1
-.if defined(WITH_GDB_LINK)
+#.if defined(WITH_GDB_LINK)
+.if ${PORT_OPTIONS:MGDB_LINK)
 	${LN} -sf gdb${VER} ${PREFIX}/bin/gdb
 .endif
-.if defined(WITH_PYTHON)
+#.if defined(WITH_PYTHON)
+.if ${PORT_OPTIONS:MPYTHON}
 	(cd ${WRKSRC}/gdb; ${GMAKE} install-python )
 	(cd ${WRKSRC}/gdb/data-directory; ${GMAKE} install-python )
 .endif
diff -rNu devel/gdb.orig/files/fbsd-threads.c devel/gdb/files/fbsd-threads.c
--- devel/gdb.orig/files/fbsd-threads.c	2012-01-27 10:38:15.000000000 +0100
+++ devel/gdb/files/fbsd-threads.c	2012-06-20 14:19:23.000000000 +0200
@@ -439,6 +439,10 @@
   if (td_ta_new_p == NULL)
     return;
 
+  /* Don't try to attach to a dead target if there is no core file. */
+  if (!target_has_execution && core_bfd == NULL)
+    return;
+
   /* Nothing to do.  The thread library was already detected and the
      target vector was already activated.  */
   if (fbsd_thread_active)


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->scheidell 
Responsible-Changed-By: scheidell 
Responsible-Changed-When: Fri Jun 22 20:04:01 UTC 2012 
Responsible-Changed-Why:  
I'll take it. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=169265 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/169265: commit references a PR
Date: Sat, 23 Jun 2012 09:12:17 +0000 (UTC)

 scheidell    2012-06-23 09:12:05 UTC
 
   FreeBSD ports repository
 
   Modified files:
     devel/gdb            Makefile 
     devel/gdb/files      fbsd-threads.c 
   Log:
   - Fix Segfault [1]
   - Update to OptionsNG [2]
   
   PR:             ports/168444 [1]
   PR:             ports/169265 [2]
   Reported by:    bgmoser@codexterous.com [1]
   Submitted by:   Luca Pizzamiglio <luca.pizzamiglio@gmail.com> (maintainer)
   
   Revision  Changes    Path
   1.18      +14 -13    ports/devel/gdb/Makefile
   1.4       +5 -1      ports/devel/gdb/files/fbsd-threads.c
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: open->closed 
State-Changed-By: scheidell 
State-Changed-When: Sat Jun 23 10:00:14 UTC 2012 
State-Changed-Why:  
Committed. Thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=169265 
>Unformatted:
