Opened on 01/20/19 at 17:44:12
Last modified on 01/24/23 at 15:13:44
#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)
Change History (10)
comment:1 Changed on 03/07/19 at 23:57:23 by Joel Sherrill
Changed on 05/04/20 at 17:49:14 by eshan dhawan
Attachment: | 0001-Removed-aio.h-form-cpukit-include-and-added-to-newli.patch added |
---|
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
Attachment: | 0001-adding-aio.h-file-to-newlib-libc-include.patch added |
---|
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: | Indefinite → 6.1 |
---|
comment:3 Changed on 05/04/20 at 19:55:21 by eshan dhawan
Milestone: | 6.1 → 5.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.h → Move 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: | new → assigned |
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.2 → 6.1 |
---|
comment:8 Changed on 01/24/23 at 15:13:44 by Joel Sherrill
Milestone: | 6.1 → 7.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.
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.