tree 7b7a3e0effc11c4e53ea4242c3324b89a08c208c
parent 5149408a6c2500cb1701880575c41640d4fd1c2f
author twigg <twigg@google.com> 1636495588 -0800
committer fweikert <fwe@google.com> 1637246845 +0100

Properly account for StarlarkOptions at their default (=null) when calculating ST-hash

Previously, the hash calculation was skipping including StarlarkOptions that happened to be at their default values.

This is wrong since those values may still be in "affected by Starlark transition" (because either the commandline set them and the Starlark transition reset them to their Starlark defaults thus still requiring a hash change OR the commandline did not set them but a series of Starlark transitions did an default->B->default anyways causing the Starlark option to still be 'stuck' in "affected by Starlark transition").

Resolves https://github.com/bazelbuild/bazel/issues/14239

PiperOrigin-RevId: 408701552