public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* AC_GNU_SOURCE in bfd/
@ 2005-05-05  4:26 Hans-Peter Nilsson
  2005-05-05  5:06 ` H. J. Lu
  0 siblings, 1 reply; 2+ messages in thread
From: Hans-Peter Nilsson @ 2005-05-05  4:26 UTC (permalink / raw)
  To: binutils

This is what I committed, after testing native on
x86_64-unknown-linux-gnu (FC4t2), on the i686-pc-linux-gnu
(RH 7.3) autotester, and cross to cris-axis-elf (this one with
the sysdep.h reversion, not included in this email).

The only ramifications I noticed was that a bunch of
"checking whether x is declared... no"
turned "yes", and of course that -D_GNU_SOURCE isn't specified
on the command line (but in config.h).

	* Makefile.am (INCLUDES): Don't -D_GNU_SOURCE here.
	* configure.in: Call AC_GNU_SOURCE here, after AC_PROG_CC.
	* configure, config.in, Makefile.in: Regenerate.
	* sysdep.h (stpcpy): Revert last change.

Index: Makefile.am
===================================================================
RCS file: /cvs/src/src/bfd/Makefile.am,v
retrieving revision 1.153
diff -p -u -r1.153 Makefile.am
--- Makefile.am	28 Apr 2005 23:02:05 -0000	1.153
+++ Makefile.am	5 May 2005 01:47:34 -0000
@@ -598,7 +598,7 @@ BFD_LIBS = @bfd_libs@
 BFD_MACHINES = @bfd_machines@
 TDEFAULTS = @tdefaults@
 
-INCLUDES = -D_GNU_SOURCE @HDEFINES@ @COREFLAG@ @TDEFINES@ $(CSEARCH) $(CSWITCHES) -I$(srcdir)/../intl -I../intl
+INCLUDES = @HDEFINES@ @COREFLAG@ @TDEFINES@ $(CSEARCH) $(CSWITCHES) -I$(srcdir)/../intl -I../intl
 
 # C source files that correspond to .o's.
 SOURCE_CFILES = \
Index: configure.in
===================================================================
RCS file: /cvs/src/src/bfd/configure.in,v
retrieving revision 1.179
diff -p -u -r1.179 configure.in
--- configure.in	4 May 2005 22:15:30 -0000	1.179
+++ configure.in	5 May 2005 01:47:42 -0000
@@ -70,6 +70,7 @@ bfd_default_target_size=32
 # host stuff:
 
 AC_PROG_CC
+AC_GNU_SOURCE
 
 ALL_LINGUAS="fr tr ja es sv da zh_CN ro rw"
 CY_GNU_GETTEXT

brgds, H-P

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: AC_GNU_SOURCE in bfd/
  2005-05-05  4:26 AC_GNU_SOURCE in bfd/ Hans-Peter Nilsson
@ 2005-05-05  5:06 ` H. J. Lu
  0 siblings, 0 replies; 2+ messages in thread
From: H. J. Lu @ 2005-05-05  5:06 UTC (permalink / raw)
  To: Hans-Peter Nilsson; +Cc: binutils

On Thu, May 05, 2005 at 05:46:56AM +0200, Hans-Peter Nilsson wrote:
> This is what I committed, after testing native on
> x86_64-unknown-linux-gnu (FC4t2), on the i686-pc-linux-gnu
> (RH 7.3) autotester, and cross to cris-axis-elf (this one with
> the sysdep.h reversion, not included in this email).
> 
> The only ramifications I noticed was that a bunch of
> "checking whether x is declared... no"
> turned "yes", and of course that -D_GNU_SOURCE isn't specified
> on the command line (but in config.h).
> 
> 	* Makefile.am (INCLUDES): Don't -D_GNU_SOURCE here.
> 	* configure.in: Call AC_GNU_SOURCE here, after AC_PROG_CC.

Can you check out doc/Makefile.am? You may need to do

# make install

to see -D_GNU_SOURCE in bfd/doc.

Thanks.


H.J.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-05-05  5:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-05-05  4:26 AC_GNU_SOURCE in bfd/ Hans-Peter Nilsson
2005-05-05  5:06 ` H. J. Lu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).