Add MPI build matrix - Githubissues

conda-forge / netcdf4-feedstock

A conda-smithy repository for netcdf4.

BSD 3-Clause "New" or "Revised" License

3 stars 25 forks source link

Add MPI build matrix #95

Closed xylar closed 4 years ago

xylar commented 4 years ago

Also adds a parallel test.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

closes #43

conda-forge-linter commented 4 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.

xylar commented 4 years ago

@conda-forge-admin, please rerender

xylar commented 4 years ago

After rerendering, the feedstock is pulling in libnetcdf 4.7.2 rather than 4.7.1. @conda-forge/netcdf4, any suggestions on how to keep this pinned correctly but still pull in the right MPI version?

xylar commented 4 years ago

@conda-forge-admin, please rerender

conda-forge-linter commented 4 years ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to re-render for you, but it looks like there was nothing to do.

xylar commented 4 years ago

Answering my own question from above: https://github.com/conda-forge/h5py-feedstock/blob/master/recipe/meta.yaml#L49-L52 It looks like the solution to getting pinnings is to list each dependency twice.

xylar commented 4 years ago

@conda-forge/netcdf4, I think we're there. Anyone want to do some final testing before I merge this?

ocefpaf commented 4 years ago

@xylar this is great! Thanks so much for doing this. I can see all the PRs that lead you here and it was not easy!

xylar commented 4 years ago

@ocefpaf, thanks again!