{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/cli/issues/181","labels_url":"https://api.github.com/repos/dotnet/cli/issues/181/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/cli/issues/181/comments","events_url":"https://api.github.com/repos/dotnet/cli/issues/181/events","html_url":"https://github.com/dotnet/cli/issues/181","id":116495183,"number":181,"title":"Cmd escaping is causing argument parse errors","user":{"login":"weshaggard","id":9010698,"avatar_url":"https://avatars.githubusercontent.com/u/9010698?v=3","gravatar_id":"","url":"https://api.github.com/users/weshaggard","html_url":"https://github.com/weshaggard","followers_url":"https://api.github.com/users/weshaggard/followers","following_url":"https://api.github.com/users/weshaggard/following{/other_user}","gists_url":"https://api.github.com/users/weshaggard/gists{/gist_id}","starred_url":"https://api.github.com/users/weshaggard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/weshaggard/subscriptions","organizations_url":"https://api.github.com/users/weshaggard/orgs","repos_url":"https://api.github.com/users/weshaggard/repos","events_url":"https://api.github.com/users/weshaggard/events{/privacy}","received_events_url":"https://api.github.com/users/weshaggard/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dotnet/cli/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":{"login":"krwq","id":660048,"avatar_url":"https://avatars.githubusercontent.com/u/660048?v=3","gravatar_id":"","url":"https://api.github.com/users/krwq","html_url":"https://github.com/krwq","followers_url":"https://api.github.com/users/krwq/followers","following_url":"https://api.github.com/users/krwq/following{/other_user}","gists_url":"https://api.github.com/users/krwq/gists{/gist_id}","starred_url":"https://api.github.com/users/krwq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krwq/subscriptions","organizations_url":"https://api.github.com/users/krwq/orgs","repos_url":"https://api.github.com/users/krwq/repos","events_url":"https://api.github.com/users/krwq/events{/privacy}","received_events_url":"https://api.github.com/users/krwq/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/dotnet/cli/milestones/6","html_url":"https://github.com/dotnet/cli/milestones/Beta-Jan","labels_url":"https://api.github.com/repos/dotnet/cli/milestones/6/labels","id":1411110,"number":6,"title":"Beta-Jan","description":"","creator":{"login":"Petermarcu","id":8228359,"avatar_url":"https://avatars.githubusercontent.com/u/8228359?v=3","gravatar_id":"","url":"https://api.github.com/users/Petermarcu","html_url":"https://github.com/Petermarcu","followers_url":"https://api.github.com/users/Petermarcu/followers","following_url":"https://api.github.com/users/Petermarcu/following{/other_user}","gists_url":"https://api.github.com/users/Petermarcu/gists{/gist_id}","starred_url":"https://api.github.com/users/Petermarcu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Petermarcu/subscriptions","organizations_url":"https://api.github.com/users/Petermarcu/orgs","repos_url":"https://api.github.com/users/Petermarcu/repos","events_url":"https://api.github.com/users/Petermarcu/events{/privacy}","received_events_url":"https://api.github.com/users/Petermarcu/received_events","type":"User","site_admin":false},"open_issues":35,"closed_issues":14,"state":"open","created_at":"2015-11-13T21:27:13Z","updated_at":"2015-12-02T09:04:32Z","due_on":"2016-01-16T08:00:00Z","closed_at":null},"comments":10,"created_at":"2015-11-12T07:26:14Z","updated_at":"2015-12-02T19:33:42Z","closed_at":null,"body":"https://github.com/dotnet/cli/blob/master/src/Microsoft.DotNet.Cli.Utils/Command.cs#L64\r\nThe escaping of quotes appear to be unnecessary in some scenarios.\r\n\r\n```\r\n> .\\dotnet -v publish ..\\test -f dnxcore50 -r win7-x64\r\nPublishing test for DNXCore,Version=v5.0/win7-x64\r\n> C:\\Windows\\system32\\cmd.exe /C \"dotnet-compile --framework \\\"DNXCore,Version=v5.0\\\" --output \\\"C:\\temp\\dotnet_test\\tes\r\nt\\bin\\Debug\\dnxcore50\\win7-x64\\\" --configuration \\\"Debug\\\" \\\"C:\\temp\\dotnet_test\\test\\\"\"\r\nSystem.ArgumentNullException: Value cannot be null.\r\nParameter name: frameworkVersion\r\n   at NuGet.Frameworks.NuGetFramework..ctor(String frameworkIdentifier, Version frameworkVersion, String frameworkProfil\r\ne)\r\n   at NuGet.Frameworks.NuGetFramework.ParseFrameworkName(String frameworkName, IFrameworkNameProvider mappings)\r\n   at NuGet.Frameworks.NuGetFramework.Parse(String folderName, IFrameworkNameProvider mappings)\r\n   at NuGet.Frameworks.NuGetFramework.Parse(String folderName)\r\n   at Microsoft.DotNet.Tools.Compiler.Program.<>c__DisplayClass0_1.<Main>b__1(String f) in D:\\git\\cli\\src\\Microsoft.DotN\r\net.Tools.Compiler\\Program.cs:line 54\r\n   at System.Linq.Enumerable.SelectListIterator`2.MoveNext()\r\n   at Microsoft.DotNet.Tools.Compiler.Program.<>c__DisplayClass0_0.<Main>b__0() in D:\\git\\cli\\src\\Microsoft.DotNet.Tools\r\n.Compiler\\Program.cs:line 56\r\n   at Microsoft.Dnx.Runtime.Common.CommandLine.CommandLineApplication.Execute(String[] args) in C:\\Users\\Wes\\.dnx\\packag\r\nes\\Microsoft.Extensions.CommandLineUtils.Sources\\1.0.0-rc2-16169\\shared\\CommandLineApplication.cs:line 263\r\n   at Microsoft.DotNet.Tools.Compiler.Program.Main(String[] args) in D:\\git\\cli\\src\\Microsoft.DotNet.Tools.Compiler\\Prog\r\nram.cs:line 70\r\n< C:\\Windows\\system32\\cmd.exe /C \"dotnet-compile --framework \\\"DNXCore,Version=v5.0\\\" --output \\\"C:\\temp\\dotnet_test\\tes\r\nt\\bin\\Debug\\dnxcore50\\win7-x64\\\" --configuration \\\"Debug\\\" \\\"C:\\temp\\dotnet_test\\test\\\"\" exited with 1 in 267 ms.\r\n```\r\n\r\nNote that we only go down this code path when the command is not on the path, if it is on the path then the command works with the following command line:\r\n\r\n```\r\n< dotnet-compile --framework \"DNXCore,Version=v5.0\" --output \"C:\\temp\\dotnet_test\\test\\bin\\Debug\\dnxcore50\\win7-x64\" --c\r\nonfiguration \"Debug\" \"C:\\temp\\dotnet_test\\test\" exited with 0 in 2575 ms.\r\n```\r\n\r\nI would submit a PR to remove the escaping but I wanted to understand if there are any cases where it will be necessary. @davidfowl @anurse any history on why this was added?"},"comment":{"url":"https://api.github.com/repos/dotnet/cli/issues/comments/161408446","html_url":"https://github.com/dotnet/cli/issues/181#issuecomment-161408446","issue_url":"https://api.github.com/repos/dotnet/cli/issues/181","id":161408446,"user":{"login":"krwq","id":660048,"avatar_url":"https://avatars.githubusercontent.com/u/660048?v=3","gravatar_id":"","url":"https://api.github.com/users/krwq","html_url":"https://github.com/krwq","followers_url":"https://api.github.com/users/krwq/followers","following_url":"https://api.github.com/users/krwq/following{/other_user}","gists_url":"https://api.github.com/users/krwq/gists{/gist_id}","starred_url":"https://api.github.com/users/krwq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krwq/subscriptions","organizations_url":"https://api.github.com/users/krwq/orgs","repos_url":"https://api.github.com/users/krwq/repos","events_url":"https://api.github.com/users/krwq/events{/privacy}","received_events_url":"https://api.github.com/users/krwq/received_events","type":"User","site_admin":false},"created_at":"2015-12-02T19:33:42Z","updated_at":"2015-12-02T19:33:42Z","body":"does it ever do?"}}