Notice: We have migrated to GitLab launching 2024-05-01 see here: https://gitlab.rtems.org/

#3676 assigned enhancement

Move aio.h from RTEMS to newlib

Reported by: deuteriumoxide Owned by: Joel Sherrill
Priority: normal Milestone: 7.1
Component: tool/newlib Version:
Severity: normal Keywords: POSIX-Compliance
Cc: Blocked By:
Blocking: #2833

Description (last modified by Joel Sherrill)

aio.h is one of the few POSIX headers that is provided by RTEMS and not newlib. The goal of this ticket is to move the header to newlib and remove it from RTEMS.

Unfortunately, patches to do this have been posted on the mailing list during the release 5 branching period. We do not want a disruption on the tools which impact all RTEMS builds to show up right now.

This requires a bit of discussion. Moving the file for 5 would require a newlib bump which is likely desirable to be easy assuming 5 has a long lifespan. We could add a patch to rtems5 which removes aio.h from newlib after it is added. This would allow bumping newlib versions with no discontinuity.

However, adding it to newlib and using it for the rtems6 toolchain is a given.

Attachments (2)

0001-Removed-aio.h-form-cpukit-include-and-added-to-newli.patch (5.7 KB) - added by eshan dhawan on 05/04/20 at 17:49:14.
This patch removes aio.h from rtems. Only the headder file is removed the implementation is still present in rtems
0001-adding-aio.h-file-to-newlib-libc-include.patch (5.2 KB) - added by eshan dhawan on 05/04/20 at 19:26:28.
this is the newlib patch for adding aio.h to newlib/libc/include

Download all attachments as: .zip

Change History (10)

comment:1 Changed on 03/07/19 at 23:57:23 by Joel Sherrill

aio.h is in RTEMS. It is cpukit/include/aio.h.

This is one of the POSIX header files in RTEMS not in newlib. Maybe it needs to get moved.

Changed on 05/04/20 at 17:49:14 by eshan dhawan

This patch removes aio.h from rtems. Only the headder file is removed the implementation is still present in rtems

Changed on 05/04/20 at 19:26:28 by eshan dhawan

this is the newlib patch for adding aio.h to newlib/libc/include

comment:2 Changed on 05/04/20 at 19:42:28 by eshan dhawan

Milestone: Indefinite6.1

comment:3 Changed on 05/04/20 at 19:55:21 by eshan dhawan

Milestone: 6.15.2

we need to decide how to manage this change through 5.2 and 6.x. If we decide it only applies to 6 in the end, we may need a patch to remove it from newlib for the 5.x series.

comment:4 Changed on 05/04/20 at 22:16:30 by Joel Sherrill

Description: modified (diff)
Summary: Add support for aio.hMove aio.h from RTEMS to newlib

comment:5 Changed on 11/09/22 at 22:15:16 by Chris Johns

Owner: set to Joel Sherrill
Status: newassigned

Has this been taken care of in 6?

comment:6 Changed on 12/01/22 at 14:52:54 by Joel Sherrill

Blocking: 2833 added

comment:7 Changed on 12/01/22 at 14:53:25 by Joel Sherrill

Milestone: 5.26.1

comment:8 Changed on 01/24/23 at 15:13:44 by Joel Sherrill

Milestone: 6.17.1

Changing milestone because there are at least two other headers which need to move and this will be a breaking tool change close to the 6 branch being made.

Note: See TracTickets for help on using tickets.