public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Fix Solaris Ada bootstrap
@ 2011-08-03 10:40 Rainer Orth
  2011-08-03 13:51 ` Arnaud Charlet
  0 siblings, 1 reply; 2+ messages in thread
From: Rainer Orth @ 2011-08-03 10:40 UTC (permalink / raw)
  To: gcc-patches; +Cc: Arnaud Charlet

Two of the latest Ada patches broke bootstrap on Solaris:

In file included from /vol/gcc/src/hg/trunk/local/gcc/ada/link.c:42:0:
./auto-host.h:1829:0: error: "_FILE_OFFSET_BITS" redefined [-Werror]
/var/gcc/regression/trunk/10-gcc/build/./prev-gcc/include-fixed/sys/feature_tests.h:197:0: note: this is the location of the previous definition
cc1plus: all warnings being treated as errors

auto-host.h has

/* Number of bits in a file offset, on hosts where this is settable. */
#ifndef USED_FOR_TARGET
#define _FILE_OFFSET_BITS 64
#endif

while <sys/feature_tests.h> defaults to 32 if undefined.  This can
easily be fixed by including auto-host.h before any system header.

i386-pc-solaris2.10 bootstrap is just in progress, but already past the
point of the previous failure.

Ok for mainline?

Thanks.
	Rainer


2011-08-03  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	* link.c: Include "auto-host.h" before system headers.

diff --git a/gcc/ada/link.c b/gcc/ada/link.c
--- a/gcc/ada/link.c
+++ b/gcc/ada/link.c
@@ -36,12 +36,12 @@
 extern "C" {
 #endif
 
-#include <string.h>
-
 #ifdef IN_GCC
 #include "auto-host.h"
 #endif
 
+#include <string.h>
+
 /*  objlist_file_supported is set to 1 when the system linker allows        */
 /*  response file, that is a file that contains the list of object files.   */
 /*  This is useful on systems where the command line length is limited,     */

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

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

* Re: Fix Solaris Ada bootstrap
  2011-08-03 10:40 Fix Solaris Ada bootstrap Rainer Orth
@ 2011-08-03 13:51 ` Arnaud Charlet
  0 siblings, 0 replies; 2+ messages in thread
From: Arnaud Charlet @ 2011-08-03 13:51 UTC (permalink / raw)
  To: Rainer Orth; +Cc: gcc-patches

> Ok for mainline?
> 
> Thanks.
> 	Rainer
> 
> 
> 2011-08-03  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
> 
> 	* link.c: Include "auto-host.h" before system headers.

OK, thanks.

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

end of thread, other threads:[~2011-08-03 13:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-03 10:40 Fix Solaris Ada bootstrap Rainer Orth
2011-08-03 13:51 ` Arnaud Charlet

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).