StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Stop changes to BAZEL_METRICS_DIR regenerating ninja file

Browse Source 
Bug: 262738679
Test: BAZEL_METRICS_DIR=dir1 m nothing
      BAZEL_METRICS_DIR=dir2 m nothing
      # Make sure that the second does not regenerate the ninja file.
Change-Id: If30f13eee5d27d902c894af04af38078f437b6df
This commit is contained in:
Paul Duffin
2022-12-21 15:55:33 +00:00
parent d1bc69d5dd
commit 184366a18f
1 changed files with 17 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
android/bazel_handler.go
View File

@@ -437,16 +437,26 @@ func NewBazelContext(c *config) (BazelContext, error) {
vars := []struct {
name string
ptr *string
// True if the environment variable needs to be tracked so that changes to the variable
// cause the ninja file to be regenerated, false otherwise. False should only be set for
// environment variables that have no effect on the generated ninja file.
track bool
}{
{"BAZEL_HOME", &paths.homeDir},
{"BAZEL_PATH", &paths.bazelPath},
{"BAZEL_OUTPUT_BASE", &paths.outputBase},
{"BAZEL_WORKSPACE", &paths.workspaceDir},
{"BAZEL_METRICS_DIR", &paths.metricsDir},
{"BAZEL_DEPS_FILE", &paths.bazelDepsFile},
{"BAZEL_HOME", &paths.homeDir, true},
{"BAZEL_PATH", &paths.bazelPath, true},
{"BAZEL_OUTPUT_BASE", &paths.outputBase, true},
{"BAZEL_WORKSPACE", &paths.workspaceDir, true},
{"BAZEL_METRICS_DIR", &paths.metricsDir, false},
{"BAZEL_DEPS_FILE", &paths.bazelDepsFile, true},
}
for _, v := range vars {
if s := c.Getenv(v.name); len(s) > 1 {
if v.track {
if s := c.Getenv(v.name); len(s) > 1 {
*v.ptr = s
continue
}
} else if s, ok := c.env[v.name]; ok {
*v.ptr = s
} else {
missing = append(missing, v.name)