conda-forge / openmpi-feedstock

A conda-smithy repository for openmpi.
BSD 3-Clause "New" or "Revised" License
9 stars 25 forks source link

ENH add external build for linking against system MPI libs #95

Closed beckermr closed 2 years ago

beckermr commented 2 years ago

This PR adds a dummy package with lower solver priority to allow for users at HPC systems to link their packages in conda compiled against openmpi to the system openmpi libraries. This is a requirement for most HPC systems. The other major mpi distribution in conda-forge, mpich already supports this. See PR https://github.com/conda-forge/mpich-feedstock/pull/44.

We'll need to change the docs which mentions this for mpich here: https://conda-forge.org/docs/user/tipsandtricks.html#using-external-message-passing-interface-mpi-libraries

Closes #90

Checklist

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

beckermr commented 2 years ago

@conda-forge-admin rerender

minrk commented 2 years ago

Is it possible to get a skip so it isn't rebuilding the same empty package on all platform combinations?

beckermr commented 2 years ago

The empty package is not the same since it is on different platforms. I worry the possible difference in solver priority between noarch and platforms would mess with track features. I don't think this is a good idea.

minrk commented 2 years ago

Fair enough. I think using noarch: generic would be the right way to go for the empty package.

beckermr commented 2 years ago

Thanks @minrk. Sorry for the quick merge here. If you’d like to be using noarch here, we can pick that up in a new PR.

minrk commented 2 years ago

Not a big deal! The build matrix is just getting big, especially with cuda compilers now. I might have a go the next time I'm bothered waiting for some builds to complete, but waiting-for-build time is the only real cost.