Conversation
dc1b85f to
2f2fb8f
Compare
|
Seems to follow the intended use of question: why not naming them py_cc_headers = "@rules_python//python/cc:current_py_cc_headers", |
|
Not particularly attached to any name in particular but I named it based on the target you'd typically override it with i.e. https://github.com/bazel-contrib/rules_python/blob/5281261a97235099a65aaeb83b74b30d16409798/python/private/hermetic_runtime_repo_setup.bzl#L110 - the python_headers target from a specific python version e.g. |
|
After second thought, I'm not sure this change will:
ps: concerning naming, I would instead use |
Make the python sdk targets configurable so that they can be overriden with e.g.
@python_3_12//:python_headersto avoid needing to transition the current python toolchain to another version.Fixes: