Re: Include FD_SETSIZE on RTEMS to 256

[Joel Sherrill <joel@rtems.org>]

[-- Attachment #1: Type: text/plain, Size: 1141 bytes --]

On Wed, Mar 13, 2024 at 10:02 AM Corinna Vinschen <vinschen@redhat.com>
wrote:

> On Mar 13 09:35, Joel Sherrill wrote:
> > On Wed, Mar 13, 2024 at 9:21 AM Jon Turney <jon.turney@dronecode.org.uk>
> > 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.

>
>
> Corinna
>
>