Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for gcc13 to 5.x #12267

Merged
merged 4 commits into from
Jan 23, 2024
Merged

Conversation

bosilca
Copy link
Member

@bosilca bosilca commented Jan 23, 2024

Fixes #12110, #12109, #12108

The only way to get IOV_MAX (and friends) is to manually include the GCC specific
syslimits.h header (sys/syslimits.h would not work either).

Fixes open-mpi#12110

Signed-off-by: George Bosilca <bosilca@icl.utk.edu>
(cherry picked from commit bbc317f)
Manually define it based on the length of the ssize_t type.

Fixes open-mpi#12109

Signed-off-by: George Bosilca <bosilca@icl.utk.edu>
(cherry picked from commit fbea826)
Instead rely on OPAL_PATH_MAX #define. This complements the fix for open-mpi#12108

Signed-off-by: George Bosilca <bosilca@icl.utk.edu>
(cherry picked from commit e051834)
We need to discover the existence and then include sys/param.h while
using the correctly named MPL defines.

Fixes open-mpi#12108

Signed-off-by: George Bosilca <bosilca@icl.utk.edu>
(cherry picked from commit 9347a55)
@bosilca bosilca requested a review from jsquyres January 23, 2024 16:45
@github-actions github-actions bot added this to the v5.0.2 milestone Jan 23, 2024
@wenduwan
Copy link
Contributor


Run /start clean
~/CI /github/workspace/ompi/ompi
Cleaning evironment
Error from server (NotFound): pods "ompi-ci-1" not found
Error: Process completed with exit code 1.

@janjust NV CI is down again - could you help with this?

I will run AWS internal CI in the meantime.

@wenduwan
Copy link
Contributor

AWS internal CI passed. Merging.

@wenduwan wenduwan merged commit ee6cbbb into open-mpi:v5.0.x Jan 23, 2024
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants