conda-forge / ucx-split-feedstock

A conda-smithy repository for ucx-split.
BSD 3-Clause "New" or "Revised" License
0 stars 23 forks source link

Apply upstream patch to fix disabling OpenMP #174

Open jakirkham opened 5 months ago

jakirkham commented 5 months ago

OpenMP is only used by UCX in testing and benchmarking (not the library itself) as discussed in issue: https://github.com/conda-forge/ucx-split-feedstock/issues/119

However it was getting picked up by default and attempting to disable it ran into issues ( https://github.com/conda-forge/ucx-split-feedstock/pull/173#discussion_r1581715325 ). Upstream since fixed the underlying problem ( https://github.com/openucx/ucx/issues/9848 ). This applies that patch and disables OpenMP again.


Checklist

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

The patch doesn't apply cleanly. We will need to wait for the next release. Will keep this open as a reminder