mirrors/dotnet-installer
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Remove quotes from /Logger argument because the argument escaper will do it for me

Browse source
This commit is contained in:
Jeff Kluge 2016-11-02 14:43:21 -07:00
parent 29bd7bf27a
commit 6a4d49e227
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/dotnet/commands/dotnet-msbuild/MSBuildForwardingApp.cs
View file

@ -39,7 +39,7 @@ namespace Microsoft.DotNet.Tools.MSBuild
argsToForward = argsToForward.Concat(new[] argsToForward = argsToForward.Concat(new[]
{ {
$"\"/Logger:{loggerType.FullName},{loggerType.GetTypeInfo().Assembly.Location}\"" $"/Logger:{loggerType.FullName},{loggerType.GetTypeInfo().Assembly.Location}"
}); });
} }
catch (Exception) catch (Exception)

2
test/dotnet-msbuild.Tests/GivenDotnetMSBuildBuildsProjects.cs
View file

@ -85,7 +85,7 @@ namespace Microsoft.DotNet.Cli.MSBuild.Tests
allArgs.Should().NotBeNull(); allArgs.Should().NotBeNull();
allArgs.Should().Contain( allArgs.Should().Contain(
value => value.StartsWith("\"/Logger:"), value => value.IndexOf("/Logger", StringComparison.OrdinalIgnoreCase) >= 0,
"The MSBuild logger argument should be specified when telemetry is enabled."); "The MSBuild logger argument should be specified when telemetry is enabled.");
} }