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

openmpi: update to 4.1.1; add new variants heterogeneous and mpi1; enable use of published binaries for all compiler-specific subports #10852

Merged
merged 1 commit into from May 5, 2021

Conversation

mascguy
Copy link
Member

@mascguy mascguy commented Apr 30, 2021

Description

  • Update openmpi to 4.1.1. In addition to general bug fixes, upstream also fixed Big Sur ARM builds.
  • Add new variants heterogeneous and mpi1
  • Enable use of published binaries for all compiler-specific subports, to match mpich
  • Add lib dependency for zlib, and corresponding configure flag
  • Ensure target compilers are added to depends_lib

Fixes Included:

Reviewers/Maintainers:

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.12.6 16G2136
Xcode 9.2 9C40b

macOS 10.13.6 17G14019
Xcode 10.1 10B61

macOS 10.14.6 18G103
Xcode 11.3.1 11C505

macOS 10.15.6 19G2021
Xcode 12.0 12A7209

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL?
  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?

@mascguy mascguy force-pushed the mascguy-openmpi-update branch 3 times, most recently from 564a76a to b435349 Compare May 1, 2021 15:49
@mascguy
Copy link
Member Author

mascguy commented May 1, 2021

* Enable use of published binaries for all compiler-specific subports
* Add lib dependency for zlib, and corresponding configure flag
* Ensure target compilers are added to depends_lib

Fixes: https://trac.macports.org/ticket/62795
Fixes: https://trac.macports.org/ticket/62788
Fixes: https://trac.macports.org/ticket/62783
Fixes: https://trac.macports.org/ticket/62214
Fixes: https://trac.macports.org/ticket/62070
Fixes: https://trac.macports.org/ticket/60597
Fixes: https://trac.macports.org/ticket/59547
@mascguy
Copy link
Member Author

mascguy commented May 3, 2021

@mascguy
Copy link
Member Author

mascguy commented May 3, 2021

@cjones051073/anyone, please review, and let me know if you have specific concerns. It would be great to merge this within the next day or so, for the Big Sur ARM fixes alone.

@mascguy mascguy merged commit ed9c96a into macports:master May 5, 2021
@mascguy mascguy deleted the mascguy-openmpi-update branch May 5, 2021 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants