Re: Include FD_SETSIZE on RTEMS to 256

On Mar 13 11:33, Joel Sherrill wrote:
> On Wed, Mar 13, 2024 at 11:03 AM Joel Sherrill <[email protected]> wrote:
> 
> >
> >
> > On Wed, Mar 13, 2024 at 10:02 AM Corinna Vinschen <[email protected]>
> > wrote:
> >
> >> On Mar 13 09:35, Joel Sherrill wrote:
> >> > On Wed, Mar 13, 2024 at 9:21 AM Jon Turney <[email protected]
> >> >
> >> > wrote:
> >> >
> >> > > On 28/02/2024 23:02, Joel Sherrill wrote:
> >> > > > Hi
> >> > > >
> >> > > > The attached patch changes FD_SETSIZE from 64 to 256 for RTEMS.
> >> > > >
> >> > > > Is it OK to apply?
> >> > > >
> >> > >
> >> > > This uses the preprocessor directive '#elifdef',
> >> > >
> >> > > It seems like support for that was added in gcc 12.
> >> > >
> >> >
> >> > Good catch. Unfortunately, I just pushed the posted patch just before
> >> your
> >> > email arrived.
> >> >
> >> > I have posted a small patch to change this to #elif defined(__rtems__).
> >> If
> >> > someone acks it, I will push it quickly to minimize who might be
> >> impacted.
> >>
> >> You sent the wrong patch apparently.  However, if your patch just
> >> changes #elifdef X to #elif defined (X), then test it with a pre-12
> >> gcc and then push it.  Soon.
> >>
> >
> > Sorry. I am building a gcc8 toolchain now to verify the change. Hopefully,
> > it will not be too much longer.
> >
> 
> Correction pushed and tested with gcc8 and a test program.
> 
> --joel

Thanks!

Corinna