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

[P4est] Enable OpenMPI #7500

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Conversation

JoshuaLampert
Copy link
Contributor

Currently, OpenMPI is disabled for P4est_jll.

@JoshuaLampert
Copy link
Contributor Author

JoshuaLampert commented Oct 6, 2023

Everything built successfully @sloede 😳

@sloede
Copy link
Contributor

sloede commented Oct 6, 2023

Great! In that case, finalize the implementation, mark PR as ready for review, and once all tests have passed, ping @giordano with the request for merging it.

@JoshuaLampert JoshuaLampert marked this pull request as ready for review October 6, 2023 10:33
@JoshuaLampert
Copy link
Contributor Author

This should be ready to merge @giordano

@sloede
Copy link
Contributor

sloede commented Oct 6, 2023

Ah, I forgot... do we need to bump the build number manually or will that be done automatically? 🤔

@ViralBShah
Copy link
Member

The build number will get bumped automatically.

@giordano giordano merged commit 06196ed into JuliaPackaging:master Oct 6, 2023
37 checks passed
@JoshuaLampert JoshuaLampert deleted the p4est-openmpi branch October 6, 2023 12:05
amontoison pushed a commit to amontoison/Yggdrasil that referenced this pull request Nov 27, 2023
* enable OpenMPI

* remove deprecated comment
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.

4 participants