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

also escape ://

Browse source
This commit is contained in:
jonsequitur 2017-02-01 10:09:54 -08:00
parent 2ea3af799d
commit 667f1cf018
2 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -80,8 +80,10 @@ namespace Microsoft.DotNet.Tools.MSBuild
} }
private static string Escape(string arg) => private static string Escape(string arg) =>
// this is a workaround for https://github.com/Microsoft/msbuild/issues/1622
(arg.StartsWith("/p:RestoreSources=", StringComparison.OrdinalIgnoreCase)) ? (arg.StartsWith("/p:RestoreSources=", StringComparison.OrdinalIgnoreCase)) ?
arg.Replace(";", "%3B") : // <-- this is a workaround for https://github.com/Microsoft/msbuild/issues/1622 arg.Replace(";", "%3B")
.Replace("://", ":%2F%2F") :
arg; arg;
private static string GetMSBuildExePath() private static string GetMSBuildExePath()

1
test/dotnet.Tests/PackagedCommandTests.cs
View file

@ -41,7 +41,6 @@ namespace Microsoft.DotNet.Tests
}; };
} }
} }
public static IEnumerable<object[]> LibraryDependencyToolArguments public static IEnumerable<object[]> LibraryDependencyToolArguments
{ {
get get