conda-forge / esmf-feedstock

A conda-smithy repository for esmf.
BSD 3-Clause "New" or "Revised" License
2 stars 20 forks source link

Fix OpenMPI build after C++ bindings were removed #51

Closed xylar closed 4 years ago

xylar commented 4 years ago

Checklist

closes #50

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-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

This is mostly a PR to test whether anything changed with a rerender and to see if the builds pass. So far, I don't see any reason to merge it but I'm hoping it will help me figure out #50

xylar commented 4 years ago

This fixed the issue in a local build for reasons I don't fully understand. My guess is that it's to do with this: https://github.com/conda-forge/openmpi-feedstock/pull/53

xylar commented 4 years ago

@conda-forge-admin, please restart ci

xylar commented 4 years ago

Weird CI issue on OSX OpenMPI the first time. Glad it worked out on a retry. Fingers crossed it doesn't happen again after I merge...