Use mpi mutex & Add activate/deactivate scripts

conda-forge / msmpi-feedstock

A conda-smithy repository for msmpi.

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

1 stars 7 forks source link

Use mpi mutex & Add activate/deactivate scripts #1

Closed leofang closed 3 years ago

leofang commented 3 years ago

~~Blocked by https://github.com/conda-forge/mpi-feedstock/pull/7.~~ UPDATE: it's in. Context: https://github.com/conda-forge/mpi4py-feedstock/pull/33#issuecomment-754892876.

I also added activate/deactivate scripts because after installation MS-MPI would set those variables for MPI applications to use. For example, FindMPI.cmake and mpi4py would pick up those env variables.

Checklist

[x] Used a personal 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.

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

leofang commented 3 years ago

@conda-forge-admin, please rerender

isuruf commented 3 years ago

Thanks @leofang. We need to mark the older builds as broken after this is merged.

leofang commented 3 years ago

We need to mark the older builds as broken after this is merged.

Oh, I didn't think it's necessary, but would happy to send a PR as requested.

@conda-forge-admin, please rerender

github-actions[bot] commented 3 years ago

Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do.

leofang commented 3 years ago

@conda-forge-admin, please restart CI

leofang commented 3 years ago

@conda-forge-admin, please restart CI

leofang commented 3 years ago

@conda-forge/msmpi @mccarthyryanc @conda-forge/core This is ready. PTAL. Thanks.

RyanMcCarthy-NOAA commented 3 years ago

@leofang strong work and thank you! Sorry it took me so long to figure out what was going on!

leofang commented 3 years ago

No worries! Happy to help 🙂Thanks to you both @mccarthyryanc @isuruf!