Bug 169662 - [MAINTAINER] devel/gdb: Split THREADS patches to better support GNU binutils
Summary: [MAINTAINER] devel/gdb: Split THREADS patches to better support GNU binutils
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Michael Scheidell
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-05 13:40 UTC by luca.pizzamiglio
Modified: 2012-07-27 08:10 UTC (History)
0 users

See Also:


Attachments
file.diff (6.07 KB, patch)
2012-07-05 13:40 UTC, luca.pizzamiglio
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description luca.pizzamiglio 2012-07-05 13:40:02 UTC
This patch improve the management of THREADS option patches.
The source file list always includes fbsd-threads.c, even if THREADS was off. The successful build is caused by a robust implementation of ar (ignoring file not found, it is not an error).

With this patch, the file list is coherent with patches.

The previous implementation created some problem with GNU ar (provided by binutils): this ar didn't ignore missing files, blocking build.

Fix: Patch attached with submission follows:
Comment 1 Michael Scheidell freebsd_committer freebsd_triage 2012-07-26 12:46:23 UTC
Responsible Changed
From-To: freebsd-ports-bugs->scheidell

I'll take it.
Comment 2 Michael Scheidell freebsd_committer freebsd_triage 2012-07-27 08:05:37 UTC
State Changed
From-To: open->closed

Committed. Thanks!
Comment 3 dfilter service freebsd_committer freebsd_triage 2012-07-27 08:05:39 UTC
Author: scheidell
Date: Fri Jul 27 07:05:23 2012
New Revision: 301593
URL: http://svn.freebsd.org/changeset/ports/301593

Log:
  - Split THREADS patches to better support GNU binutils
  
  PR:		ports/169662
  Submitted by:	Luca Pizzamiglio <luca.pizzamiglio@gmail.com> (maintainer)

Added:
  head/devel/gdb/files/extrapatch-gdb-Makefile.in   (contents, props changed)
  head/devel/gdb/files/extrapatch-gdb-configure.tgt   (contents, props changed)
Deleted:
  head/devel/gdb/files/patch-gdb-configure.tgt
Modified:
  head/devel/gdb/Makefile
  head/devel/gdb/files/patch-gdb-Makefile.in

Modified: head/devel/gdb/Makefile
==============================================================================
--- head/devel/gdb/Makefile	Fri Jul 27 06:57:20 2012	(r301592)
+++ head/devel/gdb/Makefile	Fri Jul 27 07:05:23 2012	(r301593)
@@ -7,7 +7,7 @@
 
 PORTNAME=	gdb
 PORTVERSION=	7.4.1
-PORTREVISION=	2
+PORTREVISION=	3
 CATEGORIES=	devel
 MASTER_SITES=	GNU
 
@@ -61,6 +61,10 @@ USE_PYTHON=	2.5-2.7
 
 .include <bsd.port.pre.mk>
 
+.if ${PORT_OPTIONS:MTHREADS}
+EXTRA_PATCHES=	${FILESDIR}/extrapatch-gdb-configure.tgt ${FILESDIR}/extrapatch-gdb-Makefile.in
+.endif
+
 .if ${PORT_OPTIONS:MDEBUG}
 CFLAGS+=	-g
 .endif

Added: head/devel/gdb/files/extrapatch-gdb-Makefile.in
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/devel/gdb/files/extrapatch-gdb-Makefile.in	Fri Jul 27 07:05:23 2012	(r301593)
@@ -0,0 +1,29 @@
+--- gdb/Makefile.in.orig	2011-03-30 07:55:39.000000000 +0100
++++ gdb/Makefile.in	2011-10-29 08:02:29.600973609 +0100
+@@ -571,7 +571,7 @@
+ 	xstormy16-tdep.o \
+ 	xtensa-config.o xtensa-tdep.o xtensa-linux-tdep.o \
+ 	glibc-tdep.o \
+-	bsd-uthread.o \
++	fbsd-threads.o \
+ 	nbsd-tdep.o obsd-tdep.o \
+ 	sol2-tdep.o \
+ 	solib-frv.o solib-irix.o solib-svr4.o solib-target.o \
+@@ -782,7 +782,7 @@
+ gnulib/stdint.in.h remote.h gdb.h sparc-nat.h \
+ gdbthread.h dwarf2-frame.h dwarf2-frame-tailcall.h nbsd-nat.h dcache.h \
+ amd64-nat.h s390-tdep.h arm-linux-tdep.h exceptions.h macroscope.h \
+-gdbarch.h bsd-uthread.h gdb_stat.h memory-map.h	memrange.h \
++gdbarch.h gdb_stat.h memory-map.h	memrange.h \
+ mdebugread.h m88k-tdep.h stabsread.h hppa-linux-offsets.h linux-fork.h \
+ ser-unix.h inf-ptrace.h terminal.h ui-out.h frame-base.h \
+ f-lang.h dwarf2loc.h value.h sparc-tdep.h defs.h target-descriptions.h \
+@@ -1452,7 +1452,7 @@
+ 	armnbsd-nat.c armbsd-tdep.c armnbsd-tdep.c armobsd-tdep.c \
+ 	avr-tdep.c \
+ 	bfin-linux-tdep.c bfin-tdep.c \
+-	bsd-uthread.c bsd-kvm.c \
++	fbsd-threads.c bsd-kvm.c \
+ 	core-regset.c corelow.c \
+ 	dcache.c dicos-tdep.c darwin-nat.c \
+ 	exec.c \

Added: head/devel/gdb/files/extrapatch-gdb-configure.tgt
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/devel/gdb/files/extrapatch-gdb-configure.tgt	Fri Jul 27 07:05:23 2012	(r301593)
@@ -0,0 +1,20 @@
+--- gdb/configure.tgt.orig	2010-09-30 13:38:12.841406000 -0400
++++ gdb/configure.tgt	2010-09-30 13:38:49.131127000 -0400
+@@ -169,7 +169,7 @@
+ i[34567]86-*-freebsd* | i[34567]86-*-kfreebsd*-gnu)
+ 	# Target: FreeBSD/i386
+ 	gdb_target_obs="i386-tdep.o i387-tdep.o i386bsd-tdep.o i386fbsd-tdep.o \
+-			bsd-uthread.o corelow.o solib.o solib-svr4.o"
++			fbsd-threads.o corelow.o solib.o solib-svr4.o"
+ 	;;
+ i[34567]86-*-netbsd* | i[34567]86-*-knetbsd*-gnu)
+ 	# Target: NetBSD/i386
+@@ -579,7 +579,7 @@
+ 	# Target: FreeBSD/amd64
+ 	gdb_target_obs="amd64-tdep.o amd64fbsd-tdep.o i386-tdep.o \
+ 			i387-tdep.o i386bsd-tdep.o i386fbsd-tdep.o \
+-			bsd-uthread.o corelow.o solib.o solib-svr4.o"
++			fbsd-threads.o corelow.o solib.o solib-svr4.o"
+ 	;;
+ x86_64-*-mingw*)
+         # Target: MingW/amd64

Modified: head/devel/gdb/files/patch-gdb-Makefile.in
==============================================================================
--- head/devel/gdb/files/patch-gdb-Makefile.in	Fri Jul 27 06:57:20 2012	(r301592)
+++ head/devel/gdb/files/patch-gdb-Makefile.in	Fri Jul 27 07:05:23 2012	(r301593)
@@ -1,14 +1,5 @@
 --- gdb/Makefile.in.orig	2011-03-30 07:55:39.000000000 +0100
 +++ gdb/Makefile.in	2011-10-29 08:02:29.600973609 +0100
-@@ -571,7 +571,7 @@
- 	xstormy16-tdep.o \
- 	xtensa-config.o xtensa-tdep.o xtensa-linux-tdep.o \
- 	glibc-tdep.o \
--	bsd-uthread.o \
-+	fbsd-threads.o \
- 	nbsd-tdep.o obsd-tdep.o \
- 	sol2-tdep.o \
- 	solib-frv.o solib-irix.o solib-svr4.o solib-target.o \
 @@ -758,7 +758,7 @@
  osf-share/cma_deb_core.h osf-share/AT386/cma_thread_io.h \
  osf-share/cma_sched.h proc-utils.h arm-tdep.h ax-gdb.h ppcnbsd-tdep.h \
@@ -18,15 +9,6 @@
  i386bsd-nat.h xml-support.h xml-tdesc.h alphabsd-tdep.h gdb_obstack.h \
  ia64-tdep.h ada-lang.h varobj.h frv-tdep.h nto-tdep.h serial.h \
  c-lang.h d-lang.h frame.h event-loop.h block.h cli/cli-setshow.h	\
-@@ -782,7 +782,7 @@
- gnulib/stdint.in.h remote.h gdb.h sparc-nat.h \
- gdbthread.h dwarf2-frame.h dwarf2-frame-tailcall.h nbsd-nat.h dcache.h \
- amd64-nat.h s390-tdep.h arm-linux-tdep.h exceptions.h macroscope.h \
--gdbarch.h bsd-uthread.h gdb_stat.h memory-map.h	memrange.h \
-+gdbarch.h gdb_stat.h memory-map.h	memrange.h \
- mdebugread.h m88k-tdep.h stabsread.h hppa-linux-offsets.h linux-fork.h \
- ser-unix.h inf-ptrace.h terminal.h ui-out.h frame-base.h \
- f-lang.h dwarf2loc.h value.h sparc-tdep.h defs.h target-descriptions.h \
 @@ -1161,7 +1161,7 @@
  # Removing the old gdb first works better if it is running, at least on SunOS.
  gdb$(EXEEXT): gdb.o libgdb.a $(ADD_DEPS) $(CDEPS) $(TDEPLIBS)
@@ -36,12 +18,3 @@
  		-o gdb$(EXEEXT) gdb.o libgdb.a \
  		$(TDEPLIBS) $(TUI_LIBRARY) $(CLIBS) $(LOADLIBES)
  
-@@ -1452,7 +1452,7 @@
- 	armnbsd-nat.c armbsd-tdep.c armnbsd-tdep.c armobsd-tdep.c \
- 	avr-tdep.c \
- 	bfin-linux-tdep.c bfin-tdep.c \
--	bsd-uthread.c bsd-kvm.c \
-+	fbsd-threads.c bsd-kvm.c \
- 	core-regset.c corelow.c \
- 	dcache.c dicos-tdep.c darwin-nat.c \
- 	exec.c \
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"