Fabian Meumertzheim comments

Results 1673 comments of


                                            Fabian Meumertzheim

data attributes are not propagated from objc_library to cc_library or cc_binary

Looking at `objc_library.bzl`, it doesn't seem to be doing anything with `ctx.attr.data` and instead only returns the output archives as `data_runfiles` (which also seems unnecessary).

Avoid deduplicating linkopts of cc_shared_library dependencies.

@bazel-io fork 9.0.0

platform exec_properties keys with period silently ignored

I've run into this before, but I don't see an easy solution: The available exec groups vary between rules and there are valid use cases for specifying platform properties for...

[wrapper] Make `cc_wrapper.sh` POSIX-compliant and use `#!/bin/sh`

This is a case in which Copilot comments are highly relevant and much more thorough than what I could provide. Please take a look.

`hdrs` includes are passed as `-isystem` to clang, suppressing important warnings

This will be fixed in Bazel itself: https://github.com/bazelbuild/bazel/pull/25750

Figure out a plan to stabilize isolated extension usages

Just noting some insights I had that are relevant for the evaluation: 1. Isolated extension usages receive independent lockfile entries, which limits the impact of tag changes for non-deterministic extensions....

Figure out a plan to stabilize isolated extension usages

@alexeagle We should be careful with experimental flags. Their entire point is to have users opt into the risk of being broken if the behavior of the flag changes :-)...

Figure out a plan to stabilize isolated extension usages

Would isolated extension usage help? I guess not since they can only really share a statically enumerated list of repos. What I could see work is a version of "path...

Figure out a plan to stabilize isolated extension usages

In https://bazelbuild.slack.com/archives/C09E58X3AQ7/p1764934858912079, @malt3 and I discussed what evolved into a plan to make isolated extension usage obsolete by providing extensions with a way to implement them themselves: 1. Extend `bazel...

Figure out a plan to stabilize isolated extension usages

@dzbarsky My plan is to add an `isolated_namespace` attribute to all relevant `go_deps` tag classes. If set to `foo`, `use_repo` arguments as generated by `bazel mod tidy` would take the...