Re: Add sigaltstack() prototype - Round 2

[Corinna Vinschen <vinschen at redhat dot com>]

On Aug 18 13:37, Joel Sherrill wrote:
> 
> On 8/18/2014 1:18 PM, Corinna Vinschen wrote:
> > On Aug 18 09:04, Joel Sherrill wrote:
> >> This is now committed.  Thanks for all the reviews and comments.
> >>
> >> --joel
> > There was a problem with your patch.  All the macros and type definitions
> > are in the RTEMS-only block, while the declaration of sigaltstack is in a
> > Cygwin || RTEMS block.  This leads to errors along the lines of
> >
> >   error: âstack_tâ has not been declared
> >
> > on Cygwin.  I fixed that by guarding the sigaltstack declaration with an
> > additional #ifdef __rtems__.
> Sorry. I thought the constant values were likely to be OS specific. I
> intended the
> structure and call to be available based on the XSI/POSIX version.
> 
> It really isn't there on Cygwin so if you want to make it RTEMS
> specific, I don't
> mind.  I don't know if the rule is let it compile but not link with
> newlib for
> methods in .h files that are defined by the standard.

I guess it's ok one way or the other.


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: pgp1BusWgYsvP.pgp
Description: PGP signature