maxbartel
maxbartel
> Hmmm... points taken. I'm on the fence here. If we just want to opt the release build in to including CUDA and leave developer builds stuck with the previous...
> ll that controls is the default value? You should still be able to enable it if the "available" check fails? Ah I think I misread https://cmake.org/cmake/help/latest/module/CMakeDependentOption.html (especially the part...
I can do that. I will move CUDA to that behavior in this PR then and try to move rocm in a follow-up. We probably also need to change some...
@benvanik I updated the PR to only include CUDA for aarch64 when building the package. I would like to post a PSA on the email list before changing the default...
@benvanik Could I get another review/opinion on this PR please? :)
Sounds like we found consensus then. However it doesn't feel like this PR is the correct place to make this change. I would create a follow up PR with a...