Do not turn on AssumeCleanOutputs by default on CloudBuild

smera · smera · commit 6ea40639c309 · 2022-10-27T22:19:09.000-07:00
diff --git a/Public/Src/Engine/Dll/Engine.cs b/Public/Src/Engine/Dll/Engine.cs
@@ -1298,12 +1298,6 @@ public static bool PopulateAndValidateConfiguration(
                     mutableConfig.Schedule.MaxWorkersPerModule = mutableConfig.Distribution.BuildWorkers.Count + 1;
                     mutableConfig.Schedule.ModuleAffinityLoadFactor = 1;
                 }
-
-                if (!mutableConfig.Engine.AssumeCleanOutputs.HasValue)
-                {
-                    // We assume clean outputs in CloudBuild builds.
-                    mutableConfig.Engine.AssumeCleanOutputs = true;
-                }
             }
             else
             {

Original file line number	Diff line number	Diff line change
`@@ -1298,12 +1298,6 @@ public static bool PopulateAndValidateConfiguration(`
`1298`	`1298`	`mutableConfig.Schedule.MaxWorkersPerModule = mutableConfig.Distribution.BuildWorkers.Count + 1;`
`1299`	`1299`	`mutableConfig.Schedule.ModuleAffinityLoadFactor = 1;`
`1300`	`1300`	`}`
`1301`		`-`
`1302`		`- if (!mutableConfig.Engine.AssumeCleanOutputs.HasValue)`
`1303`		`- {`
`1304`		`- // We assume clean outputs in CloudBuild builds.`
`1305`		`- mutableConfig.Engine.AssumeCleanOutputs = true;`
`1306`		`- }`
`1307`	`1301`	`}`
`1308`	`1302`	`else`
`1309`	`1303`	`{`