Skip to content

v4.0.x: add missing #include to oshmem/shmem/c/shmem_context.c. #6564

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

Merged
merged 1 commit into from
Apr 8, 2019

Conversation

benmenadue
Copy link
Contributor

This file uses MCA_SPML_CALL but doesn't include the header that defines this macro.

Fixes: #6562

Signed-off-by: Ben Menadue [email protected]

Signed-off-by: Ben Menadue <[email protected]>
(cherry picked from commit 063596b)
@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@hppritcha hppritcha added this to the v4.0.2 milestone Apr 8, 2019
@gpaulsen gpaulsen merged commit db1cb30 into open-mpi:v4.0.x Apr 8, 2019
@jsquyres
Copy link
Member

jsquyres commented Apr 9, 2019

@gpaulsen @hppritcha Is there a reason this PR (and #6563) didn't go through the full CI tests?

@gpaulsen
Copy link
Member

gpaulsen commented Apr 9, 2019

Hmmm. I thought it had.

@jsquyres
Copy link
Member

jsquyres commented Apr 9, 2019

This PR came from someone outside the Github organization, so the OMPI Jenkins CI doesn't submit it for full testing unless an org member says "ok to test". It's a DOS prevention mechanism.

The Mellanox Jenkins doesn't use this mechanism and tests everything.

@benmenadue benmenadue deleted the v4.0.x-fix-shmem-context branch November 28, 2019 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants