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

Merge "Limit environment during ninja"

Browse Source
This commit is contained in:
Dan Willemsen
2020-01-04 17:51:03 +00:00
committed by Gerrit Code Review
parent 68534a29d6 260db53da3
commit 2de4152e61
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
ui/build/ninja.go
View File

@@ -93,7 +93,9 @@ func runNinja(ctx Context, config Config) {
// //
// For the majority of cases, either Soong or the makefiles should be replicating any // For the majority of cases, either Soong or the makefiles should be replicating any
// necessary environment variables in the command line of each action that needs it. // necessary environment variables in the command line of each action that needs it.
if cmd.Environment.IsFalse("ALLOW_NINJA_ENV") { if cmd.Environment.IsEnvTrue("ALLOW_NINJA_ENV") {
ctx.Println("Allowing all environment variables during ninja; incremental builds may be unsafe.")
} else {
cmd.Environment.Allow(append([]string{ cmd.Environment.Allow(append([]string{
"ASAN_SYMBOLIZER_PATH", "ASAN_SYMBOLIZER_PATH",
"HOME", "HOME",