Re: ADA support for Debian GNU/Hurd

[Svante Signell]

On Sun, 2012-04-01 at 22:16 +0200, Samuel Thibault wrote:
> Svante Signell, le Fri 30 Mar 2012 12:55:17 +0200, a écrit :
> > --- a/src/gcc/ada/gcc-interface_Makefile.in 2011-11-18 12:46:58.000000000 
> > +0100
> > +++ b/src/gcc/ada/gcc-interface/Makefile.in 2012-03-28 11:28:09.000000000 
> > +0200
> > @@ -1143,6 +1116,32 @@
> >    LIBRARY_VERSION := $(LIB_VERSION)
> >  endif
> >  
> > +ifeq ($(strip $(filter-out %86 gnu%,$(arch) $(osys))),)
> > +  LIBGNAT_TARGET_PAIRS = \
> > +  a-intnam.ads<a-intnam-freebsd.ads \
> > +  a-numaux.adb<a-numaux-x86.adb \
> > +  a-numaux.ads<a-numaux-x86.ads \
> > +  s-inmaop.adb<s-inmaop-posix.adb \
> > +  s-intman.adb<s-intman-posix.adb \
> > +  s-osinte.adb<s-osinte-kfreebsd-gnu.adb \
> 
> It seems odd to be using the kfreebsd-gnu file here. Isn't the gnu
> version below
> 
> > +  s-osinte.ads<s-osinte-gnu.ads \
> 
> defining everything needed?

.ads is the specification file, and .adb contains the body code.
Previously I used the kfreebsd and posix versions at both places. There
was no difference in behaviour. Will take a look at
s-osinte-kfreebsd-gnu.adb file, Maybe that one is causing the tasking
problems. (I think I looked into that file, and did not find any reason
to create a gnu version)

> > +  s-osprim.adb<s-osprim-posix.adb \
> > +  s-taprop.adb<s-taprop-gnu.adb \
> > +  s-taspri.ads<s-taspri-posix.ads \
> > +  s-tpopsp.adb<s-tpopsp-posix-foreign.adb \
> > +  system.ads<system-freebsd-x86.ads
> 
> Here as well, it'd be better to introduce a system-gnu-x86.ads, even if
> it's almost the same as the freebsd version.

Maybe you are right, I just wanted to minimize the number of new files
and started with the posix versions. But as an example with
s-tparop-posix.adb I had to remove the SCHED_FIFO and Ceiling Locking
parts, and ended up with the *-gnu.adb file.