We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
strip_include_prefix
Thank you for providing these nice rules for cuda. It would be great if the cuda_library and cuda_object would provide and implement the strip_include_prefix attribute and the rules_cc do: https://bazel.build/reference/be/c-cpp#cc_library.strip_include_prefix
cuda_library
cuda_object
rules_cc
The text was updated successfully, but these errors were encountered:
We were looking into the repo to see how we could implement the strip_include_prefix. Checking how rules_cc does it gave us https://bazel.build/versions/6.2.0/rules/lib/cc_common#compile, which has the strip_include_prefix as attribute. For this repo, it looks like you guys implemented your own compile function in https://github.com/bazel-contrib/rules_cuda/blob/main/cuda/private/actions/compile.bzl, hence I think it would make sense to add this attribute there.
compile
Could you give some advise if we are on the right track here or wheter you would implement this differently?
Sorry, something went wrong.
Yeah, if we are going to implement it, we should align it with cc_* rules.
For now, you can wrap you cuh headers with a cc_library first to achieve the purpose.
No branches or pull requests
Thank you for providing these nice rules for cuda. It would be great if the
cuda_library
andcuda_object
would provide and implement thestrip_include_prefix
attribute and therules_cc
do: https://bazel.build/reference/be/c-cpp#cc_library.strip_include_prefixThe text was updated successfully, but these errors were encountered: