Merge pull request #292 from RevenantX/master

Fix setting up PreprocessorSymbols for MSBuild project.
OmniSharp · Aug 27, 2015 · 2ae98ab · 2ae98ab
2 parents 71db63f + 7fc033b
commit 2ae98ab
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/scripts/Omnisharp.cmd b/scripts/Omnisharp.cmd
@@ -1,3 +1,3 @@
 SETLOCAL
 
-%USERPROFILE%\.dnx\runtimes\dnx-clr-win-x86.1.0.0-beta4\bin\dnx %~dp0..\src\OmniSharp run %*
+"%USERPROFILE%\.dnx\runtimes\dnx-clr-win-x86.1.0.0-beta4\bin\dnx" %~dp0..\src\OmniSharp run %*
diff --git a/src/OmniSharp/MSBuild/MSBuildProjectSystem.cs b/src/OmniSharp/MSBuild/MSBuildProjectSystem.cs
@@ -243,7 +243,7 @@ private void UpdateProject(ProjectFileInfo projectFileInfo)
                     : new CSharpParseOptions();
                 if (projectFileInfo.DefineConstants != null && projectFileInfo.DefineConstants.Any())
                 {
-                    parseOptions.WithPreprocessorSymbols(projectFileInfo.DefineConstants);
+                    parseOptions = parseOptions.WithPreprocessorSymbols(projectFileInfo.DefineConstants);
                 }
                 _workspace.SetParseOptions(project.Id, parseOptions);
             }