Prevent reading from .rsp file for worker nodes

[YuliiaKovalova]

Context

Sometimes Windows Defender blocks on MSBuild.rsp file reading operation (e.g. https://dev.azure.com/devdiv/DevDiv/_workitems/edit/2308633).
Since response file content isn't needed for worker nodes, we can skip reading from t.

Changes Made

Add a check for nodemode: arg in command like

Testing

Local testing when rsp file is manually locked.

[jeffkl]

Rather than hard code the argument parsing logic to ignore one thing if another is specified, perhaps it would better to just specify the /noautoresponse command-line argument when spawning the worker node?

msbuild/src/Build/BackEnd/Components/Communications/NodeProviderOutOfProc.cs

Line 97 in f329cc4

string commandLineArgs = $"/nologo /nodemode:1 /nodeReuse:{ComponentHost.BuildParameters.EnableNodeReuse.ToString().ToLower()} /low:{ComponentHost.BuildParameters.LowPriority.ToString().ToLower()}";

[YuliiaKovalova]

Rather than hard code the argument parsing logic to ignore one thing if another is specified, perhaps it would better to just specify the /noautoresponse command-line argument when spawning the worker node?

msbuild/src/Build/BackEnd/Components/Communications/NodeProviderOutOfProc.cs

Line 97 in f329cc4

string commandLineArgs = $"/nologo /nodemode:1 /nodeReuse:{ComponentHost.BuildParameters.EnableNodeReuse.ToString().ToLower()} /low:{ComponentHost.BuildParameters.LowPriority.ToString().ToLower()}";

Thank you for the nice tip! I haven't known about this switch before.