[xbuild] Disable pre-2.0 csc hack to use /debug:portable (#4322)
authorAnkit Jain <radical@gmail.com>
Fri, 3 Feb 2017 12:19:05 +0000 (07:19 -0500)
committerAlexander Köplinger <alex.koeplinger@outlook.com>
Fri, 3 Feb 2017 12:19:05 +0000 (13:19 +0100)
commit5e27431a9ac461f52e12e3162cb9f93cbf71f3ea
treea154fe4f50d08d3cb6cf5ee71e210d4d573d8d97
parent227eadaee68b2f6e62dcf92cdf6b6bb40c7104ad
[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.
mcs/class/Microsoft.Build.Tasks/Microsoft.Build.Tasks/ManagedCompiler.cs