conda-forge / nvtx-feedstock

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

Use cuda-nvtx. #13

Closed bdice closed 1 year ago

bdice commented 1 year ago

Checklist

This PR uses the new package cuda-nvtx instead of nvtx-c as the source of the NVTX headers. We ignore the run_exports of that package, to avoid introducing a __cuda metapackage dependency on the driver version. This nvtx package is independent of CUDA version and supports CPU-only as well.

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

bdice commented 1 year ago

Closing this. After some offline discussion with @jakirkham we no longer think this rebuild is necessary. It is fine to use nvtx built on nvtx-c alongside cuda-nvtx. The packages won't clobber files because cuda-nvtx puts its contents into a targets/ directory for CUDA, and nvtx-c only provides header-only components so there can't be library conflicts.