michaelboyd2
michaelboyd2
We are currently working around this with this patch: ``` --- python/generate.go +++ python/generate.go @@ -375,7 +375,7 @@ func (py *Python) GenerateRules(args language.GenerateArgs) language.GenerateRes addModuleDependencies(deps). generateImportsAttribute() } - if (hasPyTestEntryPointFile...
Support make variable expansion in environment variables when using python_register_multi_toolchains
At the moment, I am using this patch to work around this issue: ``` --- python/config_settings/transition.bzl +++ python/config_settings/transition.bzl @@ -58,6 +58,10 @@ def _transition_py_impl(ctx): ) env = {} for k,...
Support make variable expansion in environment variables when using python_register_multi_toolchains
This still needs doing. In fact, Bazel 7 is out now so perhaps a proper fix can be made. I am thinking of having a go at this using expand_location_and_make_variables...
I feel like this patch fixes the immediate issue: ``` --- python/pip_install/tools/dependency_resolver/dependency_resolver.py +++ python/pip_install/tools/dependency_resolver/dependency_resolver.py @@ -25,7 +25,7 @@ import click import piptools.writer as piptools_writer from piptools.scripts.compile import cli -from python.runfiles...
It is also possible to fix with `legacy_create_init = True` but I feel this shouldn't be needed? This [issue](https://github.com/bazelbuild/bazel/issues/10076) implies that this will not be an option long term (although...