conda-forge / graphblas-feedstock

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

Remove reference to deprecated openmp package. Use llvm-openmp instead. #22

Closed arthurp closed 2 years ago

arthurp commented 2 years ago

Checklist

The openmp package appears to be deprecated and has not been updated since version 8. Other packages have switched to llvm-openmp, so do the same. This allows us to build against LLVM 12.

Also this avoids confusing conflicts caused by the fact that this package effectively required LLVM 8, but other packages require recent LLVM versions. The result was that if graphblas was installed many packages would be forced to very old versions.

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.

arthurp commented 2 years ago

@conda-forge-admin, please rerender

github-actions[bot] commented 2 years ago

Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like I wasn't able to push to the master branch of KatanaGraph/graphblas-feedstock. Did you check the "Allow edits from maintainers" box?

NOTE: PRs from organization accounts cannot be rerendered because of GitHub permissions.

eriknw commented 2 years ago

LGTM, thanks @arthurp !