Re: [PATCH] sys/types.h: Avoid bit-manipulation of signed values

[Corinna Vinschen <vinschen at redhat dot com>]

On Nov 27 19:08, Nick Withers wrote:
> On Fri, 2015-11-27 at 09:03 +0100, Sebastian Huber wrote:
> > 
> > On 27/11/15 08:46, Nick Withers wrote:
> > > On Fri, 2015-11-27 at 08:26 +0100, Sebastian Huber wrote:
> > > > > 
> > > > > On 26/11/15 10:17, Corinna Vinschen wrote:
> > > > > > > > > > > Looks good. I need some time to use this in RTEMS,
> > > > > > > > > > > since I am
> > > > > > > > > > > currently busy
> > > > > > > > > > > > > > > with other stuff and this is only a nice to
> > > > > > > > > > > > > > > have.
> > > > > > > > > > > 
> > > > > > > > > > > That's ok, I'm pretty busy myself.  Can you ping
> > > > > > > > > > > back as soon
> > > > > > > > > > > as you
> > > > > > > > > > > tested this?
> > > > > > > Did you test this in the meantime?
> > > > > 
> > > > > Sorry, please don't wait for me. It will be next year before I
> > > > > can
> > > > > work
> > > > > on this.

I'm sorry to read that.

> > > Is this the kind of thing a humble RTEMS user might be able to
> > > figure
> > > out?
> > > 
> > > It sounds like I'd have to do some hacking on / remove RTEMS'
> > > sys/select.h at least?
> > 
> > The goal is to use a Newlib <sys/select.h> that is capable to replace
> > these two files:
> > 
> > https://git.rtems.org/rtems/tree/cpukit/libnetworking/sys/select.h
> > https://git.rtems.org/rtems-libbsd/tree/freebsd/sys/sys/select.h
> > 
> > What complicates the things a bit is that the FreeBSD file uses
> > 
> > https://git.rtems.org/rtems-libbsd/tree/freebsd/sys/sys/_sigset.h
> > 
> > which is used by other header files as well.
> 
> Righto, thanks Sebastian.
> 
> I'm heading off for the weekend, but if I don't hear anything I'll have
> a crack at it on Monday.

The first thing would be to check if sys/select.h from Cygwin is
sufficent for RTEMS as well, provided you apply my patch from
https://sourceware.org/ml/newlib/2015/msg00832.html and move the
resulting winsup/cygwin/include/sys/select.h to newlib/libc/include/sys.

If that works fine, I apply my patch.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: pgpzDTijdnfL0.pgp
Description: PGP signature