Hi! On Mon, 18 Jun 2012 12:23:47 +0200, Arnaud Charlet <[email protected]> wrote: > > Here is a patch that adds a basic GCC Ada/GNAT configuration for x86 > > GNU/Hurd. This has originally been created by Svante Signell for use > > with Debian's GCC 4.6 package, and now I have ported it to trunk, and > > also applied some further changes. > > > > 2012-06-16 Svante Signell <[email protected]> > > Thomas Schwinge <[email protected]> > > > > gcc/ada/ > > * gcc-interface/Makefile.in: Add configuration for x86 GNU/Hurd. > > * gsocket.h [__GNU__] (Has_Sockaddr_Len): #define to 1. > > * s-oscons-tmplt.c: Add checks for __GNU__ next to several __linux__ > > and linux conditionals. > > * terminals.c [__GNU__]: Don't #include <termio.h>. #define USE_GETPT. > > * s-osinte-gnu.ads: New file. > > * s-taprop-gnu.adb: Likewise. > > Well, the main question is: who is volunteering to maintain this code > and fix it when things break?
I'm available; I also maintain other GNU/Hurd toolchain bits.
> Also, is s-taprop-gnu.adb really necessary? Assuming GNU Hurd provides
> a posix pthread interface, then you should be using s-taprop-posix.adb,
> NOT s-taprop-gnu.adb
Thanks for the suggestion, I'll look into that.
> Another comment:
>
> > --- a/gcc/ada/s-oscons-tmplt.c
> > +++ b/gcc/ada/s-oscons-tmplt.c
> > @@ -80,7 +80,7 @@ pragma Style_Checks ("M32766");
> >
> > /* Feature macro definitions */
> >
> > -#if defined (__linux__) && !defined (_XOPEN_SOURCE)
> > +#if (defined (__linux__) || defined (__GNU__)) && !defined
> > (_XOPEN_SOURCE)
> > /** For Linux _XOPEN_SOURCE must be defined, otherwise IOV_MAX is not
> > defined
> > **/
> > #define _XOPEN_SOURCE 500
>
> You need to update the comment here, since the section so far only
> applied to GNU/Linux and not GNU/Hurd.
In fact, should that perhaps (unverified) simply say »For glibc,
_XOPEN_SOURCE must be defined [...]« -- or is this code meant to be
usable on GNU/Linux with a C library different from glibc?
> So assuming the above comment update, my main objection is to introduce
> a s-taprop-gnu.adb which will cause maintenance troubles.
>
> If you use s-taprop-posix.adb instead and you or someone else volunteers
> for maintaining these files, then this could work.
Ack, thanks.
Grüße,
Thomas
pgp2Vwx2JeoZB.pgp
Description: PGP signature
