raulcd commented on code in PR #50279: URL: https://github.com/apache/arrow/pull/50279#discussion_r3498208668
########## dev/tasks/linux-packages/apache-arrow/debian/control.in: ########## @@ -30,7 +30,11 @@ Build-Depends: meson, ninja-build, nlohmann-json-dev | nlohmann-json3-dev, - nvidia-cuda-toolkit [!arm64], +# nvidia-cuda-toolkit may be unavailable (e.g. removed from Debian forky). +# The pkg.apache-arrow.nocuda build profile (set via DEB_BUILD_PROFILES, e.g. +# in the debian-forky Dockerfile) skips this build dependency and the CUDA +# packages below. + nvidia-cuda-toolkit [!arm64] <!pkg.apache-arrow.nocuda>, Review Comment: I think that's fine ... as we are controlling whether nvidia-cuda-toolkit is installed or not. Also if it starts breaking because nvidia-cuda-toolkit is now installed on debian-forky might be a good sign to remove the build profile so might not be bad to have a "notification" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
