Jon Brandvein
Jon Brandvein
Discussed offline and we agreed that kwargs should remain mutable. It looks only a very small percentage of .bzl files match a regex looking for assignments to `kwargs` keys, but...
Note that for Bazel, function hashability is also related to concerns around serializing and fingerprinting/digesting functions. The details of which are not all paged into my head right now...
@bazel-io flag 8.0.0
@wyverald There's three CLs to be cherrypicked imminently that'll close off this issue.
@bazel-io fork 8.0.0
It's infeasible to fix this in time for Bazel 8.0. It requires not just an update to the `visible()` query function's implementation, but also a change to the `QueryVisibility` machinery...
Blocked in part on #7844.
Looks like there's some background on the windows workaround in #3717.
Let's track the synthetic attribute part separately as FR #23893, without necessarily blocking 8.0 on that part. (The rest of this issue still blocks 8.0.)
@bazel-io fork 8.0.0