Skip to content

Commit

Permalink
Cleanly split BinDir to RuntimeBinDir and TestBinDir (#42829)
Browse files Browse the repository at this point in the history
  • Loading branch information
trylek authored Oct 2, 2020
1 parent 0a9a939 commit a986fe1
Show file tree
Hide file tree
Showing 21 changed files with 146 additions and 150 deletions.
4 changes: 2 additions & 2 deletions src/coreclr/dir.common.props
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,12 @@
<BaseIntermediateOutputPath>$(RepoRoot)artifacts\obj\coreclr\$(MSBuildProjectName)\</BaseIntermediateOutputPath>

<SourceDir>$(ProjectDir)src\</SourceDir>
<BinDir>$(ArtifactsDir)bin\coreclr\$(PlatformConfigPathPart)\</BinDir>
<RuntimeBinDir>$(ArtifactsDir)bin\coreclr\$(PlatformConfigPathPart)\</RuntimeBinDir>

<!-- We don't append back slash because this path is used by nuget.exe as output directory and it
fails to write packages to it if the path contains the forward slash.
-->
<PackagesBinDir>$(BinDir).nuget\</PackagesBinDir>
<PackagesBinDir>$(RuntimeBinDir).nuget\</PackagesBinDir>
</PropertyGroup>

<PropertyGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
</PropertyGroup>

<ItemGroup>
<NativeBinary Include="$(BinDir)ilasm$(ApplicationFileExtension)" />
<NativeBinary Include="$(RuntimeBinDir)ilasm$(ApplicationFileExtension)" />
</ItemGroup>

<Import Project="$([MSBuild]::GetPathOfFileAbove(Directory.Build.targets))" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
</PropertyGroup>

<ItemGroup>
<NativeBinary Include="$(BinDir)ildasm$(ApplicationFileExtension)" />
<NativeBinary Include="$(RuntimeBinDir)ildasm$(ApplicationFileExtension)" />
</ItemGroup>

<Import Project="$([MSBuild]::GetPathOfFileAbove(Directory.Build.targets))" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
</PropertyGroup>

<ItemGroup>
<NativeBinary Include="$(BinDir)corerun$(ApplicationFileExtension)" />
<NativeBinary Include="$(RuntimeBinDir)corerun$(ApplicationFileExtension)" />
</ItemGroup>

<Import Project="$([MSBuild]::GetPathOfFileAbove(Directory.Build.targets))" />
Expand Down
4 changes: 2 additions & 2 deletions src/coreclr/src/Directory.Build.targets
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@
AfterTargets="Build"
Condition="Exists(@(BuiltBinary -> '%(RootDir)%(Directory)%(Filename).pdb'))"
Inputs="@(BuiltBinary -> '%(RootDir)%(Directory)%(Filename).pdb')"
Outputs="@(BuiltBinary -> '$(BinDir)PDB/%(Filename).pdb')">
Outputs="@(BuiltBinary -> '$(RuntimeBinDir)PDB/%(Filename).pdb')">

<Move SourceFiles="@(BuiltBinary -> '%(RootDir)%(Directory)%(Filename).pdb')"
DestinationFolder="$(BinDir)PDB" />
DestinationFolder="$(RuntimeBinDir)PDB" />

</Target>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<DebugSymbols>true</DebugSymbols>

<!-- Force System.Private.CoreLib.dll into a special IL output directory -->
<OutputPath>$(BinDir)/IL/</OutputPath>
<OutputPath>$(RuntimeBinDir)/IL/</OutputPath>
<Configurations>Debug;Release;Checked</Configurations>
<Platforms>x64;x86;arm;arm64</Platforms>

Expand Down
2 changes: 1 addition & 1 deletion src/coreclr/src/pal/tests/palsuite/producepaltestlist.proj
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<FilteredTestList Remove="@(ExcludeList)" />
</ItemGroup>

<WriteLinesToFile File="$(BinDir)paltests/paltestlist.txt"
<WriteLinesToFile File="$(RuntimeBinDir)paltests/paltestlist.txt"
Lines="@(FilteredTestList)"
Overwrite="true"
WriteOnlyWhenDifferent="true"/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<CLSCompliant>false</CLSCompliant>
<NoWarn>8002,NU1701</NoWarn>
<RuntimeIdentifiers>win-x64;win-x86</RuntimeIdentifiers>
<OutputPath>$(BinDir)</OutputPath>
<OutputPath>$(RuntimeBinDir)</OutputPath>
<Platforms>AnyCPU;x64</Platforms>
<PlatformTarget>AnyCPU</PlatformTarget>
<LangVersion>7.3</LangVersion>
Expand Down
20 changes: 10 additions & 10 deletions src/coreclr/src/tools/aot/crossgen2/crossgen2.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<PlatformTarget>AnyCPU</PlatformTarget>
<AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
<AppendTargetFrameworkToOutputPath Condition="'$(BuildingInsideVisualStudio)' == 'true'">true</AppendTargetFrameworkToOutputPath>
<OutputPath>$(BinDir)/crossgen2</OutputPath>
<OutputPath>$(RuntimeBinDir)/crossgen2</OutputPath>
<GenerateRuntimeConfigurationFiles>true</GenerateRuntimeConfigurationFiles>
<EnableDefaultEmbeddedResourceItems>false</EnableDefaultEmbeddedResourceItems>
<RuntimeIdentifiers>linux-x64;linux-musl-x64;win-x64</RuntimeIdentifiers>
Expand Down Expand Up @@ -61,13 +61,13 @@
</PropertyGroup>

<ItemGroup>
<Content Include="$(BinDir)\$(JitInterfaceLibraryName)"
<Content Include="$(RuntimeBinDir)\$(JitInterfaceLibraryName)"
CopyToOutputDirectory="PreserveNewest"
CopyToPublishDirectory="PreserveNewest"
Link="%(FileName)%(Extension)"
/>

<Content Include="$(BinDir)\$(NativeArchFolder)$(LibraryNamePrefix)clrjit_*_$(TargetArchitecture)$(LibraryNameExtension)"
<Content Include="$(RuntimeBinDir)\$(NativeArchFolder)$(LibraryNamePrefix)clrjit_*_$(TargetArchitecture)$(LibraryNameExtension)"
CopyToOutputDirectory="PreserveNewest"
CopyToPublishDirectory="PreserveNewest"
Link="%(FileName)%(Extension)"
Expand All @@ -77,7 +77,7 @@
<!-- On windows we can re-use the clrjit.dll produced in the build for aot compilation. On Linux
this works at runtime, but makes it difficult to debug the jit.-->
<ItemGroup Condition="'$(TargetOS)' == 'Windows_NT'">
<Content Include="$(BinDir)\$(NativeArchFolder)$(LibraryNamePrefix)clrjit$(LibraryNameExtension)"
<Content Include="$(RuntimeBinDir)\$(NativeArchFolder)$(LibraryNamePrefix)clrjit$(LibraryNameExtension)"
CopyToOutputDirectory="PreserveNewest"
CopyToPublishDirectory="PreserveNewest"
Link="$(LibraryNamePrefix)clrjit_$(TargetSpec)$(LibraryNameExtension)"
Expand All @@ -87,13 +87,13 @@
<Target Name="CreateCrossTargetingPackage" AfterTargets="Build" Condition="'$(CrossHostArch)' != ''">

<PropertyGroup>
<CrossPackageFolder>$(BinDir)\$(CrossHostArch)\crossgen2</CrossPackageFolder>
<CrossPackageFolder>$(RuntimeBinDir)\$(CrossHostArch)\crossgen2</CrossPackageFolder>
</PropertyGroup>

<ItemGroup>
<PackageFile Include="$(BinDir)\crossgen2\*"
Exclude="$(BinDir)\crossgen2\$(JitInterfaceLibraryName);$(BinDir)\crossgen2\$(LibraryNamePrefix)clrjit_*$(LibraryNameExtension)" />
<PackageFile Include="$(BinDir)\$(CrossHostArch)\$(LibraryNamePrefix)jitinterface_$(CrossHostArch)$(LibraryNameExtension)" />
<PackageFile Include="$(RuntimeBinDir)\crossgen2\*"
Exclude="$(RuntimeBinDir)\crossgen2\$(JitInterfaceLibraryName);$(RuntimeBinDir)\crossgen2\$(LibraryNamePrefix)clrjit_*$(LibraryNameExtension)" />
<PackageFile Include="$(RuntimeBinDir)\$(CrossHostArch)\$(LibraryNamePrefix)jitinterface_$(CrossHostArch)$(LibraryNameExtension)" />
</ItemGroup>

<MakeDir Directories="$(CrossPackageFolder)" />
Expand All @@ -105,13 +105,13 @@

<Copy
Condition="'$(TargetOS)' != 'Windows_NT'"
SourceFiles="$(BinDir)$(CrossHostArch)\$(LibraryNamePrefix)clrjit_$(CrossTargetSpec)$(LibraryNameExtension)"
SourceFiles="$(RuntimeBinDir)$(CrossHostArch)\$(LibraryNamePrefix)clrjit_$(CrossTargetSpec)$(LibraryNameExtension)"
DestinationFolder="$(CrossPackageFolder)"
UseHardLinksIfPossible="true"
/>
<Copy
Condition="'$(TargetOS)' == 'Windows_NT'"
SourceFiles="$(BinDir)$(CrossHostArch)\$(LibraryNamePrefix)clrjit$(LibraryNameExtension)"
SourceFiles="$(RuntimeBinDir)$(CrossHostArch)\$(LibraryNamePrefix)clrjit$(LibraryNameExtension)"
DestinationFiles="$(CrossPackageFolder)\$(LibraryNamePrefix)clrjit_$(CrossTargetSpec)$(LibraryNameExtension)"
UseHardLinksIfPossible="true"
/>
Expand Down
2 changes: 1 addition & 1 deletion src/coreclr/src/tools/dotnet-pgo/dotnet-pgo.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<TargetFramework>$(NetCoreAppCurrent)</TargetFramework>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<LangVersion>8.0</LangVersion>
<OutputPath>$(BinDir)/dotnet-pgo</OutputPath>
<OutputPath>$(RuntimeBinDir)/dotnet-pgo</OutputPath>
<GenerateRuntimeConfigurationFiles>true</GenerateRuntimeConfigurationFiles>
<EnableDefaultEmbeddedResourceItems>false</EnableDefaultEmbeddedResourceItems>
<RuntimeIdentifiers>linux-x64;linux-musl-x64;win-x64</RuntimeIdentifiers>
Expand Down
2 changes: 1 addition & 1 deletion src/coreclr/src/tools/r2rdump/R2RDump.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<CLSCompliant>false</CLSCompliant>
<NoWarn>8002,NU1701</NoWarn>
<RuntimeIdentifiers>win-x64;win-x86</RuntimeIdentifiers>
<OutputPath>$(BinDir)/R2RDump</OutputPath>
<OutputPath>$(RuntimeBinDir)/R2RDump</OutputPath>
<Platforms>AnyCPU;x64</Platforms>
</PropertyGroup>

Expand Down
2 changes: 1 addition & 1 deletion src/coreclr/src/tools/r2rtest/R2RTest.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<TargetFramework>$(NetCoreAppCurrent)</TargetFramework>
<NoWarn>8002,NU1701</NoWarn>
<PlatformTarget>AnyCPU</PlatformTarget>
<OutputPath>$(BinDir)\R2RTest</OutputPath>
<OutputPath>$(RuntimeBinDir)\R2RTest</OutputPath>
</PropertyGroup>

<ItemGroup>
Expand Down
2 changes: 1 addition & 1 deletion src/coreclr/src/tools/runincontext/runincontext.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@
<RuntimeFrameworkVersion>$(MicrosoftNETCoreAppVersion)</RuntimeFrameworkVersion>
<UseAppHost>false</UseAppHost>
<CLRTestKind>BuildOnly</CLRTestKind>
<OutputPath>$(BinDir)</OutputPath>
<OutputPath>$(RuntimeBinDir)</OutputPath>
</PropertyGroup>
</Project>
2 changes: 1 addition & 1 deletion src/mono/Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
<PropertyGroup>
<ProjectDir>$(MSBuildThisFileDirectory)</ProjectDir>
<SourceDir>$(ProjectDir)src\</SourceDir>
<BinDir>$(ArtifactsBinDir)mono\$(PlatformConfigPathPart)\</BinDir>
<RuntimeBinDir>$(ArtifactsBinDir)mono\$(PlatformConfigPathPart)\</RuntimeBinDir>

<BaseIntermediateOutputPath>$(ArtifactsObjDir)mono\$(MSBuildProjectName)\</BaseIntermediateOutputPath>
</PropertyGroup>
Expand Down
32 changes: 16 additions & 16 deletions src/mono/mono.proj
Original file line number Diff line number Diff line change
Expand Up @@ -807,11 +807,11 @@
<!-- Copy Mono runtime bits to $(Destination) -->
<Target Name="CopyMonoRuntimeFilesFromArtifactsToDestination">
<ItemGroup>
<_MonoRuntimeArtifacts Include="$(BinDir)\*.*" />
<_MonoRuntimeArtifacts Include="$(RuntimeBinDir)\*.*" />
</ItemGroup>
<Error Condition="'$(Destination)' == ''" Text="Destination should not be empty" />
<Error Condition="@(_MonoRuntimeArtifacts->Count()) &lt; 2" Text="Mono artifacts were not found at $(BinDir)" />
<Message Text="Copying Mono Runtime artifacts from '$(BinDir)' to '$(Destination)'.'" Importance="High" />
<Error Condition="@(_MonoRuntimeArtifacts->Count()) &lt; 2" Text="Mono artifacts were not found at $(RuntimeBinDir)" />
<Message Text="Copying Mono Runtime artifacts from '$(RuntimeBinDir)' to '$(Destination)'.'" Importance="High" />
<Copy SourceFiles="@(_MonoRuntimeArtifacts)"
DestinationFolder="$(Destination)"
OverwriteReadOnlyFiles="true"
Expand Down Expand Up @@ -906,38 +906,38 @@
<!-- Copy Mono runtime files to artifacts directory -->
<ItemGroup>
<_MonoRuntimeArtifacts Include="$(_MonoRuntimeFilePath)">
<Destination>$(BinDir)$(MonoFileName)</Destination>
<Destination>$(RuntimeBinDir)$(MonoFileName)</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Include="$(_MonoRuntimeStaticFilePath)">
<Destination>$(BinDir)$(MonoStaticFileName)</Destination>
<Destination>$(RuntimeBinDir)$(MonoStaticFileName)</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Include="$(_MonoAotCrossFilePath)">
<Destination>$(BinDir)cross\mono-aot-cross</Destination>
<Destination>$(RuntimeBinDir)cross\mono-aot-cross</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(MonoBundleLLVMOptimizer)' == 'true'" Include="$(MonoLLVMDir)\bin\llc">
<Destination>$(BinDir)\llc</Destination>
<Destination>$(RuntimeBinDir)\llc</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(MonoBundleLLVMOptimizer)' == 'true'" Include="$(MonoLLVMDir)\bin\opt">
<Destination>$(BinDir)\opt</Destination>
<Destination>$(RuntimeBinDir)\opt</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(TargetOS)' == 'iOS' or '$(TargetOS)' == 'tvOS'" Include="$(MonoLLVMDir)\bin\llc">
<Destination>$(BinDir)cross\llc</Destination>
<Destination>$(RuntimeBinDir)cross\llc</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(TargetOS)' == 'iOS' or '$(TargetOS)' == 'tvOS'" Include="$(MonoLLVMDir)\bin\opt">
<Destination>$(BinDir)cross\opt</Destination>
<Destination>$(RuntimeBinDir)cross\opt</Destination>
</_MonoRuntimeArtifacts>
<_MonoIncludeArtifacts Include="$(MonoObjDir)out\include\**" />
<_MonoRuntimeArtifacts Condition="'$(TargetsBrowser)' == 'true'" Include="$(MonoObjDir)out\lib\libmono-ee-interp.a">
<Destination>$(BinDir)libmono-ee-interp.a</Destination>
<Destination>$(RuntimeBinDir)libmono-ee-interp.a</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(TargetsBrowser)' == 'true'" Include="$(MonoObjDir)out\lib\libmono-icall-table.a">
<Destination>$(BinDir)libmono-icall-table.a</Destination>
<Destination>$(RuntimeBinDir)libmono-icall-table.a</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(TargetsBrowser)' == 'true'" Include="$(MonoObjDir)out\lib\libmono-ilgen.a">
<Destination>$(BinDir)libmono-ilgen.a</Destination>
<Destination>$(RuntimeBinDir)libmono-ilgen.a</Destination>
</_MonoRuntimeArtifacts>
<_MonoRuntimeArtifacts Condition="'$(TargetsBrowser)' == 'true'" Include="$(MonoObjDir)out\lib\libmono-profiler-aot.a">
<Destination>$(BinDir)libmono-profiler-aot.a</Destination>
<Destination>$(RuntimeBinDir)libmono-profiler-aot.a</Destination>
</_MonoRuntimeArtifacts>
</ItemGroup>

Expand All @@ -946,11 +946,11 @@
SkipUnchangedFiles="true" />

<Copy SourceFiles="@(_MonoIncludeArtifacts)"
DestinationFiles="@(_MonoIncludeArtifacts->'$(BinDir)include\%(RecursiveDir)%(Filename)%(Extension)')"
DestinationFiles="@(_MonoIncludeArtifacts->'$(RuntimeBinDir)include\%(RecursiveDir)%(Filename)%(Extension)')"
SkipUnchangedFiles="true"
Condition="'$(TargetsiOS)' == 'true' or '$(TargetstvOS)' == 'true' or '$(TargetsAndroid)' == 'true' or '$(TargetsBrowser)' == 'true'"/>

<Exec Condition="'$(TargetsOSX)' == 'true' or '$(TargetsiOS)' == 'true' or '$(TargetstvOS)' == 'true'" Command="install_name_tool -id @rpath/$(MonoFileName) $(BinDir)$(MonoFileName)" />
<Exec Condition="'$(TargetsOSX)' == 'true' or '$(TargetsiOS)' == 'true' or '$(TargetstvOS)' == 'true'" Command="install_name_tool -id @rpath/$(MonoFileName) $(RuntimeBinDir)$(MonoFileName)" />
</Target>

<Target Name="CleanMono">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
<DebugSymbols>true</DebugSymbols>

<!-- Force System.Private.CoreLib.dll into a special IL output directory -->
<OutputPath>$(BinDir)IL/</OutputPath>
<OutputPath>$(RuntimeBinDir)IL/</OutputPath>
<Configurations>Debug;Release;Checked</Configurations>
<Platforms>x64;x86;arm;arm64;wasm</Platforms>
</PropertyGroup>
Expand Down Expand Up @@ -306,8 +306,8 @@
<Import Project="$(LibrariesProjectRoot)\System.Private.CoreLib\src\System.Private.CoreLib.Shared.projitems" Label="Shared" />

<Target Name="CopyCoreLibToBinDir" AfterTargets="Build">
<Copy SourceFiles="$(BinDir)/IL/System.Private.CoreLib.dll;$(BinDir)/IL/System.Private.CoreLib.pdb"
DestinationFolder="$(BinDir)"
<Copy SourceFiles="$(RuntimeBinDir)/IL/System.Private.CoreLib.dll;$(RuntimeBinDir)/IL/System.Private.CoreLib.pdb"
DestinationFolder="$(RuntimeBinDir)"
SkipUnchangedFiles="true" />
</Target>

Expand Down
4 changes: 1 addition & 3 deletions src/tests/Common/dir.common.props
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,7 @@

<BaseOutputPath>$(ArtifactsDir)tests\coreclr</BaseOutputPath>
<BaseOutputPathWithConfig>$([MSBuild]::NormalizePath('$(BaseOutputPath)\$(OSPlatformConfig)\'))</BaseOutputPathWithConfig>
<BinDir>$(BaseOutputPathWithConfig)</BinDir>

<TestWorkingDir>$(BaseOutputPathWithConfig)</TestWorkingDir>
<TestBinDir>$(BaseOutputPathWithConfig)</TestBinDir>

<TestSrcDir>$([MSBuild]::NormalizePath('$(RepoRoot)/src/tests/'))</TestSrcDir>
<TestSrcDir Condition="$([System.String]::new('$(MSBuildProjectDirectory)').StartsWith('$(BaseOutputPathWithConfig)'))">$(BaseOutputPathWithConfig)</TestSrcDir>
Expand Down
4 changes: 2 additions & 2 deletions src/tests/Common/dirs.proj
Original file line number Diff line number Diff line change
Expand Up @@ -122,8 +122,8 @@

<Import Project="$(MSBuildThisFileDirectory)dir.traversal.targets" />

<!-- Override clean from dir.traversal.targets and just remove the full BinDir -->
<!-- Override clean from dir.traversal.targets and just remove the full TestBinDir -->
<Target Name="Clean">
<RemoveDir Directories="$(BinDir)" />
<RemoveDir Directories="$(TestBinDir)" />
</Target>
</Project>
11 changes: 5 additions & 6 deletions src/tests/Common/helixpublishwitharcade.proj
Original file line number Diff line number Diff line change
Expand Up @@ -85,9 +85,8 @@
</PropertyGroup>

<PropertyGroup>
<BinDir>$([MSBuild]::NormalizeDirectory($(TestWorkingDir)))</BinDir>
<CoreRootDirectory>$(BinDir)Tests\Core_Root\</CoreRootDirectory>
<PayloadsRootDirectory>$(BinDir)Payloads\</PayloadsRootDirectory>
<CoreRootDirectory>$(TestBinDir)Tests\Core_Root\</CoreRootDirectory>
<PayloadsRootDirectory>$(TestBinDir)Payloads\</PayloadsRootDirectory>
<PayloadsRootDirectory>$([MSBuild]::NormalizeDirectory($(PayloadsRootDirectory)))</PayloadsRootDirectory>
<TestEnvFileName Condition=" '$(TargetsWindows)' == 'true' ">SetStressModes_$(Scenario).cmd</TestEnvFileName>
<TestEnvFileName Condition=" '$(TargetsWindows)' != 'true' ">SetStressModes_$(Scenario).sh</TestEnvFileName>
Expand Down Expand Up @@ -115,10 +114,10 @@
</_XUnitWrapperDll>
<_XUnitWrapperDll Include="@(XUnitWrapperGrouping)" />

<!-- This adds the remaining *.XUnitWrapper.dll files in BinDir unless
<!-- This adds the remaining *.XUnitWrapper.dll files in TestBinDir unless
1) they are in PayloadsRootDirectory or
2) they are grouped by XUnitWrapperGrouping. -->
<_XUnitWrapperDll Include="$(BinDir)**\*.XUnitWrapper.dll" Exclude="$(PayloadsRootDirectory)**\*.XUnitWrapper.dll;@(XUnitWrapperGrouping->Metadata('FullPath'))">
<_XUnitWrapperDll Include="$(TestBinDir)**\*.XUnitWrapper.dll" Exclude="$(PayloadsRootDirectory)**\*.XUnitWrapper.dll;@(XUnitWrapperGrouping->Metadata('FullPath'))">
<!-- Set PayloadGroup to empty string, so we can update _XUnitWrapperDll items with no PayloadGroup to default value. Unfortunatelly, we can't do this right here. -->
<PayloadGroup></PayloadGroup>
</_XUnitWrapperDll>
Expand Down Expand Up @@ -155,7 +154,7 @@
<_PayloadFiles Update="@(_PayloadFiles)">
<!-- Never use [MSBuild]::MakeRelative here! We have some files containing Unicode characters in their %(FullPath) and
MakeRelative function calls Escape function internally that replaces all the Unicode characters with %<xx>. -->
<FileRelativeToPayloadsRootDirectory>$(_PayloadGroup)\$([System.IO.Path]::GetRelativePath($(BinDir), %(FullPath)))</FileRelativeToPayloadsRootDirectory>
<FileRelativeToPayloadsRootDirectory>$(_PayloadGroup)\$([System.IO.Path]::GetRelativePath($(TestBinDir), %(FullPath)))</FileRelativeToPayloadsRootDirectory>
</_PayloadFiles>
</ItemGroup>

Expand Down
1 change: 0 additions & 1 deletion src/tests/Common/ilasm/ilasm.ilproj
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,5 @@

<PropertyGroup>
<RuntimeIdentifier>$(TargetRid)</RuntimeIdentifier>
<ProjectAssetsFile>$(TestSourceDir)Common\ilasm\obj\project.assets.json</ProjectAssetsFile>
</PropertyGroup>
</Project>
Loading

0 comments on commit a986fe1

Please sign in to comment.