conda-forge / openmpi-feedstock

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

Migrate `sysroot_*` to `{{ stdlib('c') }}` #149

Closed jakirkham closed 2 months ago

jakirkham commented 2 months ago

Follows up on this suggestion ( https://github.com/conda-forge/openmpi-feedstock/pull/147#pullrequestreview-1984041142 ), to align with broader move away from sysroot_* to {{ stdlib('c') }} ( https://github.com/conda-forge/conda-forge.github.io/issues/2102 )

cc @beckermr


Checklist

conda-forge-webservices[bot] commented 2 months 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.

jakirkham commented 2 months ago

@conda-forge-admin, please re-render

github-actions[bot] commented 2 months 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.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/openmpi-feedstock/actions/runs/8622718322.

jakirkham commented 2 months ago

Thanks Axel! 🙏

Was looking for reviews from folks like yourself 🙂

dalcinl commented 2 months ago

@jakirkham Looks like this one is ready. Should we merge?

jakirkham commented 2 months ago

If it looks good, please feel free to merge

jakirkham commented 2 months ago

Thanks all! 🙏