This is the mail archive of the mailing list for the newlib project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Add sigaltstack() prototype - Round 2

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 Vinschen
Cygwin Maintainer
Red Hat

Attachment: pgp1BusWgYsvP.pgp
Description: PGP signature

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]