Fabian Meumertzheim

Results 1692 comments of Fabian Meumertzheim

I would still consider this useful, but since https://github.com/bazelbuild/rules_go/commit/0f1e0ae83d2a6f0746b9c083880b5c2239e3256f, rules_go doesn't need it anymore. Cc @linzhp

@comius @Wyverald Looks like builtins lack protobuf in their repo mapping.

@comius told me that this is expected and will be fixed by a follow-up Bazel commit that autoloads protobuf.

Many of the new files look unused. Could we add an integration test to verify that this new feature actually works and then rely on proguard to strip out unneeded...

We actually ran into this with [Jazzer](https://github.com/CodeIntelligenceTesting/jazzer), which has a legitimate need to depend on JaCoCo during regular execution. Shading helped here: https://github.com/CodeIntelligenceTesting/jazzer/blob/main/third_party/jacoco_internal.BUILD Bazel has a built-in capability to detect...

Running with `--test_env=VERBOSE_COVERAGE=1` would help as it provides far more debug output

Bazelisk has a `--bisect` flag that automatically finds the breaking Bazel commit. Could you run that?

This is a change that I have wanted to make for a long time, thanks for sending it! I would be in favor of obtaining `cgo_context_info` from the toolchain directly,...

> The problem for Bazel though is that Java's LinkedHashMap only allows O(1) popping of oldest entries. This should no longer be a blocker with Java 21: https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/SequencedMap.html#pollLastEntry()

You can work around this by temporarily renaming the attribute: ``` buildozer 'rename path not_path' //MODULE.bazel:%local_path_override buildozer 'print module_name not_path' //MODULE.bazel:%local_path_override buildozer 'rename not_path path' //MODULE.bazel:%local_path_override ``` (using a command...