diff options
| author | Ankit Jain <radical@gmail.com> | 2017-02-03 15:19:05 +0300 |
|---|---|---|
| committer | Alexander Köplinger <alex.koeplinger@outlook.com> | 2017-02-03 15:19:05 +0300 |
| commit | 5e27431a9ac461f52e12e3162cb9f93cbf71f3ea (patch) | |
| tree | a154fe4f50d08d3cb6cf5ee71e210d4d573d8d97 /mcs/class/Microsoft.Build.Tasks | |
| parent | 227eadaee68b2f6e62dcf92cdf6b6bb40c7104ad (diff) | |
[xbuild] Disable pre-2.0 csc hack to use /debug:portable (#4322)
This hack should not be required now, as roslyn emits the correct debug
files with /debug:full and /debug:pdbonly, on non-windows platform.
This manifested as https://bugzilla.xamarin.com/show_bug.cgi?id=50710 ,
where the Xamarin.Mac targets override `$(CscToolExe)` to use `mcs`, but
xbuild's `ManagedCompiler` *always* converts `DebugType={full,pdbonly}`
to `/debug:portable`, which would cause mcs to complain.
Diffstat (limited to 'mcs/class/Microsoft.Build.Tasks')
| -rw-r--r-- | mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ManagedCompiler.cs | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ManagedCompiler.cs b/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ManagedCompiler.cs index 19eec868654..9d6404ee3eb 100644 --- a/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ManagedCompiler.cs +++ b/mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ManagedCompiler.cs @@ -58,11 +58,7 @@ namespace Microsoft.Build.Tasks { if (Bag ["CodePage"] != null) commandLine.AppendSwitchIfNotNull ("/codepage:", CodePage.ToString ()); - var dtype = DebugType; - if (string.Equals (dtype, "full", StringComparison.OrdinalIgnoreCase) || string.Equals (dtype, "pdbonly", StringComparison.OrdinalIgnoreCase)) - dtype = "portable"; - - commandLine.AppendSwitchIfNotNull ("/debug:", dtype); + commandLine.AppendSwitchIfNotNull ("/debug:", DebugType); if (Bag ["DelaySign"] != null) if (DelaySign) |