diff options
| author | Katelyn Gadd <kg@luminance.org> | 2018-08-09 03:52:37 +0300 |
|---|---|---|
| committer | Alexander Köplinger <alex.koeplinger@outlook.com> | 2018-08-09 03:52:37 +0300 |
| commit | 14d20074ffb298cc396916328897eac7f5f8c077 (patch) | |
| tree | 62244a3aa5ae6c678b4ce0fa69b438c5fb8f6ccd /mcs/class/SMDiagnostics | |
| parent | bd50f070bd693f64b2ceede91f9470ba0deb0aa1 (diff) | |
Move to generating msbuild choose elements to get if-else selection behavior for sources in projects so that we don't get erroneous duplicate files in cases where there are both profile and host platform criteria (#9952)
A recent commit revealed that in cases where we select based on a mix of host platform and profile, genproj csproj files can end up with duplicate sources because the existing <ItemGroup Condition= approach could make multiple groups match for a given compile when we really just want one.
This PR changes to generating a cascade of msbuild <Choose> elements, which give if-else selection to ensure that we only ever build a single set of files.
Diffstat (limited to 'mcs/class/SMDiagnostics')
| -rw-r--r-- | mcs/class/SMDiagnostics/SMDiagnostics.csproj | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mcs/class/SMDiagnostics/SMDiagnostics.csproj b/mcs/class/SMDiagnostics/SMDiagnostics.csproj index 392066222c0..ec170346f84 100644 --- a/mcs/class/SMDiagnostics/SMDiagnostics.csproj +++ b/mcs/class/SMDiagnostics/SMDiagnostics.csproj @@ -47,6 +47,7 @@ </PropertyGroup>
<Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
<!-- @BUILT_SOURCES@ -->
+ <!--Common files-->
<ItemGroup>
<Compile Include="..\..\build\common\SR.cs" />
<Compile Include="..\referencesource\SMDiagnostics\System\ServiceModel\Diagnostics\Activity.cs" />
@@ -70,7 +71,7 @@ <Compile Include="Assembly\AssemblyInfo.cs" />
<Compile Include="ReferenceSources\TraceSR.cs" />
</ItemGroup>
- <ItemGroup></ItemGroup>
+ <!--End of common files-->
<!-- @ALL_SOURCES@ -->
<ItemGroup>
<ProjectReference Include="../System/System.csproj" />
|