Restore $binaryLog default logic

Author: dougbu

build.ps1

@@ -349,6 +349,12 @@ if ($ForceCoreMsbuild) {
     $msbuildEngine = 'dotnet'
 }
 
+# Ensure passing neither -bl nor -nobl on CI avoids errors in tools.ps1. This is needed because both parameters are
+# $false by default i.e. they always exist. (We currently avoid binary logs but that is made visible in the YAML.)
+if ($CI -and -not $excludeCIBinarylog) {
+    $binaryLog = $true
+}
+
 # tools.ps1 corrupts global state, so reset these values in case they carried over from a previous build
 Remove-Item variable:global:_BuildTool -ea Ignore
 Remove-Item variable:global:_DotNetInstallDir -ea Ignore

build.sh

@@ -314,6 +314,12 @@ export MSBUILDDISABLENODEREUSE=1
 # Fixing this is tracked by https://github.com/dotnet/aspnetcore-internal/issues/601
 warn_as_error=false
 
+# Ensure passing neither --bl nor --nobl on CI avoids errors in tools.sh. This is needed because we set both variables
+# to false by default i.e. they always exist. (We currently avoid binary logs but that is made visible in the YAML.)
+if [[ "$ci" == true && "$exclude_ci_binary_log" == false ]]; then
+    binary_log=true
+fi
+
 # increase file descriptor limit on macOS
 if [ "$(uname)" = "Darwin" ]; then
     ulimit -n 10000