Fabian Meumertzheim
Fabian Meumertzheim
@bazel-io fork 7.4.0
@iancha1992 Added!
@rickeylev Could you take a look? I found this capability to be lacking while working on analysis tests for `cc_static_library` and noticing that using `use_default_shell_env` [has pitfalls](https://github.com/bazelbuild/bazel/issues/5980).
> I missed that this PR is waiting for me. Is there still interest to get it in? Yes. > This is exposing Action.env_inherit, which is only accessible to tests,...
The wait was worth it, this is much simpler now. @comius Getting this into 7.2.0 would make it easier to write tests for `cc_static_library`.
We could make this possibly breaking change for Bazel 8. Ideally, we would have a feature toggling this behavior so that users broken by it can easily disable it. Would...
cc @tjgq @coeuvre
> Can you repro with `--noexperimental_merged_skyframe_analysis_execution`? I can't. > Are these outputs should be downloaded (e.g. toplevel outputs, or specified by `--remote_download_regex`)? No, these outputs should not be downloaded (just...
This is the root cause: https://github.com/bazelbuild/bazel/blob/43ad74bec433c1923e2ce78605ea04cac0cdb324/src/main/java/com/google/devtools/build/lib/buildtool/ExecutionTool.java#L328-L337 Note the `RemoteArtifactChecker.IGNORE_ALL`, which means that all remote artifacts will be marked as dirty. Happy to work on this, are there any ideas of...