From 534c299946e2b32dfb2860deecd701bbb9f25055 Mon Sep 17 00:00:00 2001
From: Adeel <3840695+am11@users.noreply.github.com>
Date: Thu, 19 Nov 2020 02:34:35 +0200
Subject: [PATCH 1/3] Consolidate RID and native file naming in MSBuild scripts
* Use short variable names for native files naming convention, that are
used by `framework.sharedfx.targets` in arcade, and cleanup redefinitions
from crossgen2 and installer. e.g. `ExeSuffix` instead of
`ApplicationFileExtension`, `LibSuffix` instead of `LibraryFileExtension`
and so on. * Calculate `TargetArchitecture`, `NonPortableRuntimeOS` (for
`PortableBuild`) and `PackageRID` values once for the entire livebuild,
inside `eng/Configurations.props`. This implementation is a union of three
varied implementations that are being deleted. * Import `names.props` once in
`eng/Configurations.props` based on calculated `PackageRID` and cleanup
imports of this file from various places. * Combine OS targets definition
in MSBuild scripts.
---
Directory.Build.props | 4 +-
eng/Configurations.props | 113 +++++++++-
eng/native/init-distro-rid.sh | 10 +-
eng/native/naming.props | 53 +++--
src/coreclr/dir.common.props | 16 --
src/coreclr/src/.nuget/Directory.Build.props | 108 ----------
.../src/.nuget/Directory.Build.targets | 6 +-
.../Microsoft.NETCore.ILAsm.pkgproj | 2 +-
.../Microsoft.NETCore.ILDAsm.pkgproj | 2 +-
.../Microsoft.NETCore.TestHost.pkgproj | 2 +-
.../src/tools/aot/crossgen2/crossgen2.csproj | 24 +--
src/installer/Directory.Build.props | 204 +-----------------
.../pkg/projects/Directory.Build.props | 2 -
.../pkg/projects/Directory.Build.targets | 6 +-
.../Microsoft.NETCore.DotNetAppHost.pkgproj | 10 +-
.../Microsoft.NETCore.DotNetHost.pkgproj | 2 +-
...Microsoft.NETCore.DotNetHostPolicy.pkgproj | 2 +-
...crosoft.NETCore.DotNetHostResolver.pkgproj | 2 +-
src/installer/pkg/sfx/Directory.Build.props | 2 -
.../Microsoft.NETCore.App.Crossgen2.sfxproj | 30 +--
.../Microsoft.NETCore.App.Host.sfxproj | 10 +-
.../Microsoft.NETCore.App.Runtime.sfxproj | 12 +-
.../pkg/sfx/installers/dotnet-host.proj | 4 +-
.../pkg/sfx/installers/dotnet-hostfxr.proj | 2 +-
src/installer/tests/Directory.Build.targets | 2 +-
src/libraries/Directory.Build.props | 67 +-----
src/libraries/externals.csproj | 2 +-
src/libraries/pkg/Directory.Build.props | 2 -
.../runtime.native.System.IO.Ports.pkgproj | 2 +-
src/mono/Directory.Build.props | 18 --
src/tests/Common/ilasm/ilasm.ilproj | 2 +-
src/tests/Common/publishdependency.targets | 2 +-
.../test_dependencies.csproj | 2 +-
.../test_dependencies.fsproj | 2 +-
src/tests/Directory.Build.props | 74 -------
src/tests/build.cmd | 10 +-
src/tests/run.proj | 12 +-
37 files changed, 223 insertions(+), 602 deletions(-)
diff --git a/Directory.Build.props b/Directory.Build.props
index 799d054f3b1af..91557e1435793 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -57,7 +57,7 @@
$([MSBuild]::NormalizeDirectory('$(ArtifactsBinDir)', 'WasmAppBuilder', 'Debug', '$(NetCoreAppCurrent)', 'publish'))
$([MSBuild]::NormalizeDirectory('$(ArtifactsBinDir)', 'WasmBuildTasks', 'Debug', '$(NetCoreAppCurrent)', 'publish'))
$([MSBuild]::NormalizeDirectory('$(ArtifactsBinDir)', 'MonoAOTCompiler', 'Debug', '$(NetCoreAppCurrent)'))
-
+
$([MSBuild]::NormalizePath('$(AppleAppBuilderDir)', 'AppleAppBuilder.dll'))
$([MSBuild]::NormalizePath('$(AndroidAppBuilderDir)', 'AndroidAppBuilder.dll'))
$([MSBuild]::NormalizePath('$(WasmAppBuilderDir)', 'WasmAppBuilder.dll'))
@@ -105,8 +105,6 @@
-
- true
false
diff --git a/eng/Configurations.props b/eng/Configurations.props
index d26beec0e3e68..6f68f208fc0f5 100644
--- a/eng/Configurations.props
+++ b/eng/Configurations.props
@@ -4,6 +4,11 @@
steps and in the repository.
-->
+
+
+ true
+
+
$([MSBuild]::NormalizeDirectory('$(RepoRoot)', 'src', 'libraries'))
$([MSBuild]::NormalizeDirectory('$(RepoRoot)', 'src', 'coreclr'))
@@ -49,9 +54,6 @@
- $([System.Runtime.InteropServices.RuntimeInformation]::RuntimeIdentifier)
- win-$([System.Runtime.InteropServices.RuntimeInformation]::OSArchitecture.ToString().ToLowerInvariant)
-
OSX
FreeBSD
NetBSD
@@ -68,6 +70,109 @@
true
+
+ <_runtimeOS>$(RuntimeOS)
+
+ <_parseDistroRid>$(__DistroRid)
+ <_parseDistroRid Condition="'$(_parseDistroRid)' == ''">$([System.Runtime.InteropServices.RuntimeInformation]::RuntimeIdentifier)
+ <_distroRidIndex>$(_parseDistroRid.LastIndexOfAny("-"))
+
+ <_runtimeOS Condition="'$(_runtimeOS)' == ''">$(_parseDistroRid.SubString(0, $(_distroRidIndex)))
+
+
+ <_runtimeOS Condition="'$(TargetsMobile)' == 'true'">$(TargetOS.ToLowerInvariant())
+
+ <_runtimeOSVersionIndex>$(_runtimeOS.IndexOfAny(".-0123456789"))
+ <_runtimeOSFamily Condition="'$(_runtimeOSVersionIndex)' != '-1'">$(_runtimeOS.SubString(0, $(_runtimeOSVersionIndex)))
+
+ <_buildingInOSX>$([MSBuild]::IsOSPlatform('OSX'))
+ <_portableOS>linux
+ <_portableOS Condition="'$(_runtimeOS)' == 'linux-musl'">linux-musl
+ <_portableOS Condition="$(_buildingInOSX)">osx
+ <_portableOS Condition="'$(_runtimeOSFamily)' == 'win' or '$(TargetOS)' == 'windows'">win
+ <_portableOS Condition="'$(_runtimeOSFamily)' == 'FreeBSD'">freebsd
+ <_portableOS Condition="'$(_runtimeOSFamily)' == 'illumos'">illumos
+ <_portableOS Condition="'$(_runtimeOSFamily)' == 'Solaris'">solaris
+ <_portableOS Condition="'$(_runtimeOS)' == 'Browser'">browser
+ <_portableOS Condition="'$(_runtimeOS)' == 'ios'">ios
+ <_portableOS Condition="'$(_runtimeOS)' == 'tvos'">tvos
+ <_portableOS Condition="'$(_runtimeOS)' == 'android'">android
+
+ <_runtimeOS Condition="$(_runtimeOS.StartsWith('tizen'))">linux
+ $(_runtimeOS)
+ <_runtimeOS Condition="'$(PortableBuild)' == 'true'">$(_portableOS)
+
+
+ <_portableOS Condition="'$(TargetOS)' == 'Unix' and '$(_runtimeOSFamily)' != 'osx' and '$(_runtimeOSFamily)' != 'FreeBSD' and '$(_runtimeOS)' != 'linux-musl' and '$(_runtimeOSFamily)' != 'illumos' and '$(_runtimeOSFamily)' != 'Solaris'">linux
+
+
+
+ <_hostArch>$([System.Runtime.InteropServices.RuntimeInformation]::ProcessArchitecture.ToString().ToLowerInvariant)
+ arm
+ arm64
+ wasm
+ x64
+ x64
+
+
+
+ <_toolRuntimeRID Condition="'$(BuildingInsideVisualStudio)' == 'true'">$(_runtimeOS)-x64
+ <_toolRuntimeRID Condition="'$(_toolRuntimeRID)' == ''">$(_runtimeOS)-$(_hostArch)
+
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'linux-musl' and $(TargetArchitecture.StartsWith('arm')) and !$(_hostArch.StartsWith('arm'))">linux-x64
+
+
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'browser' and '$(TargetOS)' == 'windows'">win-x64
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'browser' and '$(TargetOS)' != 'windows' and $(_buildingInOSX)">osx-x64
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'browser' and '$(TargetOS)' != 'windows' and !$(_buildingInOSX)">linux-x64
+
+
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'android' and '$(TargetOS)' == 'windows'">win-x64
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'android' and '$(TargetOS)' != 'windows' and $(_buildingInOSX)">osx-x64
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'android' and '$(TargetOS)' != 'windows' and !$(_buildingInOSX)">linux-x64
+
+
+ <_toolRuntimeRID Condition="'$(_runtimeOS)' == 'ios' or '$(_runtimeOS)' == 'tvos'">osx-x64
+ $(_toolRuntimeRID)
+
+ <_packageRID Condition="'$(PortableBuild)' == 'true'">$(_portableOS)-$(TargetArchitecture)
+ $(_packageRID)
+ $(_runtimeOS)-$(TargetArchitecture)
+
+ <_outputRID Condition="'$(TargetOS)' == 'windows'">win-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'OSX'">osx-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'Linux'">linux-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'FreeBSD'">freebsd-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'NetBSD'">netbsd-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'illumos'">illumos-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'Solaris'">solaris-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'iOS'">ios-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'tvOS'">tvos-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'Android'">android-$(TargetArchitecture)
+ <_outputRID Condition="'$(TargetOS)' == 'Browser'">browser-$(TargetArchitecture)
+
+ $(PackageRID)
+ $(_outputRID)
+
+
+
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+ true
+
+
true
@@ -76,4 +181,6 @@
false
+
+
diff --git a/eng/native/init-distro-rid.sh b/eng/native/init-distro-rid.sh
index f5d14f2f7cd50..bddfb85f18d53 100644
--- a/eng/native/init-distro-rid.sh
+++ b/eng/native/init-distro-rid.sh
@@ -119,7 +119,6 @@ initNonPortableDistroRid()
#
# __DistroRid
# __PortableBuild
-# __RuntimeId
#
initDistroRidGlobal()
{
@@ -194,13 +193,8 @@ initDistroRidGlobal()
if [ -z "$__DistroRid" ]; then
echo "DistroRid is not set. This is almost certainly an error"
-
exit 1
- else
- echo "__DistroRid: ${__DistroRid}"
- echo "__RuntimeId: ${__DistroRid}"
-
- __RuntimeId="${__DistroRid}"
- export __RuntimeId
fi
+
+ echo "__DistroRid: ${__DistroRid}"
}
diff --git a/eng/native/naming.props b/eng/native/naming.props
index dbc7bc7cd9f88..7a02d6c6c6815 100644
--- a/eng/native/naming.props
+++ b/eng/native/naming.props
@@ -1,45 +1,54 @@
+
+ lib
+
+
-
+
- .exe
-
- .dll
- .lib
- .pdb
+ .exe
+ .dll
+ .lib
+ .pdb
-
+
- lib
- .dylib
- .a
- .dwarf
+ lib
+ .dylib
+ .a
+ .dwarf
-
+
- lib
- .so
- .a
+ lib
+ .so
+ .a
- .debug
+ .debug
- lib
- .so
- .a
- .dbg
+ lib
+ .so
+ .a
+ .dbg
-
-
+
+
+
+
+ $(SymbolsSuffix)
+
+
diff --git a/src/coreclr/dir.common.props b/src/coreclr/dir.common.props
index fbaf583697f89..08e71350e9824 100644
--- a/src/coreclr/dir.common.props
+++ b/src/coreclr/dir.common.props
@@ -50,22 +50,6 @@
$(PackageVersion)
-
-
- true
- true
- true
- true
- true
- true
- true
-
- true
-
-
- $(__DistroRid)
-
-
Portable
diff --git a/src/coreclr/src/.nuget/Directory.Build.props b/src/coreclr/src/.nuget/Directory.Build.props
index 3473cbf83b521..1e0b60ec47c4b 100644
--- a/src/coreclr/src/.nuget/Directory.Build.props
+++ b/src/coreclr/src/.nuget/Directory.Build.props
@@ -20,36 +20,14 @@
true
-
- <_parseDistroRid>$(__DistroRid)
- <_parseDistroRid Condition="'$(_parseDistroRid)' == '' and '$(TargetOS)' == 'OSX'">osx.10.12-x64
- <_distroRidIndex>$(_parseDistroRid.LastIndexOfAny("-"))
- <_archRidIndex>$([MSBuild]::Add($(_distroRidIndex), 1))
- $(_parseDistroRid.SubString(0, $(_distroRidIndex)))
- win10
-
- $(_parseDistroRid.SubString($(_archRidIndex)))
- $(Platform)
-
- $(OSRid)
-
windows;OSX;Android;Linux;FreeBSD;NetBSD;illumos;Solaris
;$(SupportedPackageOSGroups);
-
- <_runtimeOSVersionIndex>$(RuntimeOS.IndexOfAny(".-0123456789"))
- <_runtimeOSFamily Condition="'$(_runtimeOSVersionIndex)' != '-1'">$(RuntimeOS.SubString(0, $(_runtimeOSVersionIndex)))
- <_runtimeOSFamily Condition="'$(_runtimeOSVersionIndex)' == '-1'">$(RuntimeOS)
<_isSupportedOSGroup>true
- <_derivedPackageTargetOSGroup Condition="'$(_derivedPackageTargetOSGroup)' == '' and '$(_runtimeOSFamily)' == 'osx'">OSX
- <_derivedPackageTargetOSGroup Condition="'$(_derivedPackageTargetOSGroup)' == '' and '$(_runtimeOSFamily)' == 'android'">Android
- <_derivedPackageTargetOSGroup Condition="'$(_derivedPackageTargetOSGroup)' == '' and '$(_runtimeOSFamily)' == 'win'">windows
<_derivedPackageTargetOSGroup Condition="'$(_derivedPackageTargetOSGroup)' == '' and '$(TargetOS)' != ''">$(TargetOS)
<_derivedPackageTargetOSGroup Condition="'$(_derivedPackageTargetOSGroup)' == ''">Linux
@@ -73,92 +51,6 @@
true
-
-
-
-
-
- $(OutputRID)
-
-
-
-
-
- win-$(TargetArchitecture)
-
-
-
-
- osx.10.12-$(TargetArchitecture)
-
- osx-$(TargetArchitecture)
-
-
-
-
- freebsd.11-$(TargetArchitecture)
-
- freebsd-$(TargetArchitecture)
-
-
-
-
- netbsd-$(TargetArchitecture)
-
- netbsd-$(TargetArchitecture)
-
-
-
-
- illumos-$(TargetArchitecture)
-
- illumos-$(TargetArchitecture)
-
-
-
-
- solaris-$(TargetArchitecture)
-
- solaris-$(TargetArchitecture)
-
-
-
-
- android.21-$(TargetArchitecture)
-
- android-$(TargetArchitecture)
-
-
-
-
- $(OSRid)-$(TargetArchitecture)
-
- linux-$(TargetArchitecture)
-
-
-
-
- $(OSRid)-$(TargetArchitecture)
-
- linux-musl-$(TargetArchitecture)
-
-
-
-
- $(RuntimeOS)-$(TargetArchitecture)
-
-
-
-
- $(RuntimeOS)-$(TargetArchitecture)
-
- linux-$(TargetArchitecture)
-
-
-
-
-
-
diff --git a/src/coreclr/src/.nuget/Directory.Build.targets b/src/coreclr/src/.nuget/Directory.Build.targets
index f9886d59b0bbd..eeab2931cf837 100644
--- a/src/coreclr/src/.nuget/Directory.Build.targets
+++ b/src/coreclr/src/.nuget/Directory.Build.targets
@@ -9,8 +9,8 @@
Finds symbol files and injects them into the package build.
-->
-
-
+
+
@@ -42,7 +42,7 @@
-
+
diff --git a/src/coreclr/src/.nuget/Microsoft.NETCore.ILAsm/Microsoft.NETCore.ILAsm.pkgproj b/src/coreclr/src/.nuget/Microsoft.NETCore.ILAsm/Microsoft.NETCore.ILAsm.pkgproj
index ff68beebb047f..744c17ad28bb6 100644
--- a/src/coreclr/src/.nuget/Microsoft.NETCore.ILAsm/Microsoft.NETCore.ILAsm.pkgproj
+++ b/src/coreclr/src/.nuget/Microsoft.NETCore.ILAsm/Microsoft.NETCore.ILAsm.pkgproj
@@ -8,7 +8,7 @@
-
+
diff --git a/src/coreclr/src/.nuget/Microsoft.NETCore.ILDAsm/Microsoft.NETCore.ILDAsm.pkgproj b/src/coreclr/src/.nuget/Microsoft.NETCore.ILDAsm/Microsoft.NETCore.ILDAsm.pkgproj
index 116da5fd20dd2..8c6321fc7fd8c 100644
--- a/src/coreclr/src/.nuget/Microsoft.NETCore.ILDAsm/Microsoft.NETCore.ILDAsm.pkgproj
+++ b/src/coreclr/src/.nuget/Microsoft.NETCore.ILDAsm/Microsoft.NETCore.ILDAsm.pkgproj
@@ -8,7 +8,7 @@
-
+
diff --git a/src/coreclr/src/.nuget/Microsoft.NETCore.TestHost/Microsoft.NETCore.TestHost.pkgproj b/src/coreclr/src/.nuget/Microsoft.NETCore.TestHost/Microsoft.NETCore.TestHost.pkgproj
index aad674a250400..81b519f74bce8 100644
--- a/src/coreclr/src/.nuget/Microsoft.NETCore.TestHost/Microsoft.NETCore.TestHost.pkgproj
+++ b/src/coreclr/src/.nuget/Microsoft.NETCore.TestHost/Microsoft.NETCore.TestHost.pkgproj
@@ -8,7 +8,7 @@
-
+
diff --git a/src/coreclr/src/tools/aot/crossgen2/crossgen2.csproj b/src/coreclr/src/tools/aot/crossgen2/crossgen2.csproj
index b4844002483e6..e6822043a1ec9 100644
--- a/src/coreclr/src/tools/aot/crossgen2/crossgen2.csproj
+++ b/src/coreclr/src/tools/aot/crossgen2/crossgen2.csproj
@@ -63,13 +63,7 @@
$(TargetOSComponent)_$(TargetArchitecture)_$(TargetArchitecture)
$(TargetOSComponent)_$(TargetArchitecture)_$(CrossHostArch)
- lib
-
- .dll
- .so
- .dylib
-
- $(LibraryNamePrefix)jitinterface_$(TargetArchitecture)$(LibraryNameExtension)
+ $(LibPrefix)jitinterface_$(TargetArchitecture)$(LibSuffix)
@@ -79,7 +73,7 @@
Link="%(FileName)%(Extension)"
/>
-
-
@@ -104,8 +98,8 @@
-
+ Exclude="$(RuntimeBinDir)\crossgen2\$(JitInterfaceLibraryName);$(RuntimeBinDir)\crossgen2\$(LibPrefix)clrjit_*$(LibSuffix)" />
+
@@ -117,14 +111,14 @@
diff --git a/src/installer/Directory.Build.props b/src/installer/Directory.Build.props
index 28dc7f98be61f..e7bbc71a7a739 100644
--- a/src/installer/Directory.Build.props
+++ b/src/installer/Directory.Build.props
@@ -57,29 +57,6 @@
$(DefineConstants),TRACE
-
- .exe
-
-
-
- $(HostRuntimeIdentifier.Remove($(HostRuntimeIdentifier.LastIndexOf('-'))))-$(TargetArchitecture)
-
-
-
-
- win-$(TargetArchitecture)
- osx-$(TargetArchitecture)
- linux-$(TargetArchitecture)
- freebsd-$(TargetArchitecture)
- netbsd-$(TargetArchitecture)
- illumos-$(TargetArchitecture)
- solaris-$(TargetArchitecture)
- ios-$(TargetArchitecture)
- tvos-$(TargetArchitecture)
- android-$(TargetArchitecture)
- browser-$(TargetArchitecture)
-
-
$(OutputRid)
@@ -138,175 +115,22 @@
-
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
- false
+
+ true
+
+ true
-
-
-
- true
-
-
-
-
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
- true
-
-
-
-
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
- true
-
-
-
-
- true
- true
-
-
-
-
- true
- true
-
-
-
-
- true
- true
-
-
-
-
- true
- true
-
-
-
-
- true
- true
-
-
- true
-
- true
-
-
-
.zip
.tar.gz
.msi
.pkg
- .deb
- .rpm
+ .deb
+ .rpm
.exe
$(InstallerExtension)
@@ -326,16 +150,6 @@
true
-
- lib
- .so
- .dll
- .dylib
- lib
- .a
- .lib
-
-
.map
.ni.pdb
diff --git a/src/installer/pkg/projects/Directory.Build.props b/src/installer/pkg/projects/Directory.Build.props
index b9ad8328b4d29..b3d84b7fe2286 100644
--- a/src/installer/pkg/projects/Directory.Build.props
+++ b/src/installer/pkg/projects/Directory.Build.props
@@ -157,6 +157,4 @@
-
-
diff --git a/src/installer/pkg/projects/Directory.Build.targets b/src/installer/pkg/projects/Directory.Build.targets
index e29724f0f85ea..095938d03a4f8 100644
--- a/src/installer/pkg/projects/Directory.Build.targets
+++ b/src/installer/pkg/projects/Directory.Build.targets
@@ -133,8 +133,8 @@
Finds symbol files and injects them into the package build.
-->
-
-
+
+
@@ -162,7 +162,7 @@
-
+
diff --git a/src/installer/pkg/projects/Microsoft.NETCore.DotNetAppHost/Microsoft.NETCore.DotNetAppHost.pkgproj b/src/installer/pkg/projects/Microsoft.NETCore.DotNetAppHost/Microsoft.NETCore.DotNetAppHost.pkgproj
index 0ce0a5fcb2c35..eeedb0faea012 100644
--- a/src/installer/pkg/projects/Microsoft.NETCore.DotNetAppHost/Microsoft.NETCore.DotNetAppHost.pkgproj
+++ b/src/installer/pkg/projects/Microsoft.NETCore.DotNetAppHost/Microsoft.NETCore.DotNetAppHost.pkgproj
@@ -4,10 +4,10 @@
-
-
-
-
+
+
+
+
@@ -29,4 +29,4 @@
true
-
\ No newline at end of file
+
diff --git a/src/installer/pkg/projects/Microsoft.NETCore.DotNetHost/Microsoft.NETCore.DotNetHost.pkgproj b/src/installer/pkg/projects/Microsoft.NETCore.DotNetHost/Microsoft.NETCore.DotNetHost.pkgproj
index 000ba4fe48399..a957392ccde45 100644
--- a/src/installer/pkg/projects/Microsoft.NETCore.DotNetHost/Microsoft.NETCore.DotNetHost.pkgproj
+++ b/src/installer/pkg/projects/Microsoft.NETCore.DotNetHost/Microsoft.NETCore.DotNetHost.pkgproj
@@ -7,7 +7,7 @@
-
+
runtimes/$(PackageTargetRuntime)/native
true
diff --git a/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostPolicy/Microsoft.NETCore.DotNetHostPolicy.pkgproj b/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostPolicy/Microsoft.NETCore.DotNetHostPolicy.pkgproj
index 06f4c38912373..43d12c53b96dc 100644
--- a/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostPolicy/Microsoft.NETCore.DotNetHostPolicy.pkgproj
+++ b/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostPolicy/Microsoft.NETCore.DotNetHostPolicy.pkgproj
@@ -9,7 +9,7 @@
-
+
runtimes/$(PackageTargetRuntime)/native
true
diff --git a/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostResolver/Microsoft.NETCore.DotNetHostResolver.pkgproj b/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostResolver/Microsoft.NETCore.DotNetHostResolver.pkgproj
index 5a49dbb69b226..a2d33da0e18b2 100644
--- a/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostResolver/Microsoft.NETCore.DotNetHostResolver.pkgproj
+++ b/src/installer/pkg/projects/Microsoft.NETCore.DotNetHostResolver/Microsoft.NETCore.DotNetHostResolver.pkgproj
@@ -11,7 +11,7 @@
-
+
runtimes/$(PackageTargetRuntime)/native
true
diff --git a/src/installer/pkg/sfx/Directory.Build.props b/src/installer/pkg/sfx/Directory.Build.props
index a078c899d8159..6c4517eb7b5bf 100644
--- a/src/installer/pkg/sfx/Directory.Build.props
+++ b/src/installer/pkg/sfx/Directory.Build.props
@@ -20,6 +20,4 @@
true
true
-
-
diff --git a/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Crossgen2.sfxproj b/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Crossgen2.sfxproj
index ed0ed5da583db..89490af741fa2 100644
--- a/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Crossgen2.sfxproj
+++ b/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Crossgen2.sfxproj
@@ -21,7 +21,7 @@
tools/
true
-
+
unix
win
@@ -33,17 +33,17 @@
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
-
-
+
+
@@ -75,10 +75,10 @@
- <_crossTargetJit Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == '$(LibraryFilePrefix)clrjit' and '%(Extension)' == '$(LibraryFileExtension)'" />
- <_clrjit Include="@(RuntimeFiles)" Condition="'%(FileName)' == '$(LibraryFilePrefix)clrjit' and '%(Extension)' == '$(LibraryFileExtension)'" />
- <_crossTargetCrossgen Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ApplicationFileExtension)'" />
- <_crossgen Include="@(RuntimeFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ApplicationFileExtension)'" />
+ <_crossTargetJit Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == '$(LibPrefix)clrjit' and '%(Extension)' == '$(LibSuffix)'" />
+ <_clrjit Include="@(RuntimeFiles)" Condition="'%(FileName)' == '$(LibPrefix)clrjit' and '%(Extension)' == '$(LibSuffix)'" />
+ <_crossTargetCrossgen Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ExeSuffix)'" />
+ <_crossgen Include="@(RuntimeFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ExeSuffix)'" />
-
-
-
-
+
+
+
+
@@ -40,7 +40,7 @@
IsSymbolFile="true"
IsNative="true" />
<_SymbolFiles Condition="'$(TargetOS)' != 'windows'"
- Include="@(NativeRuntimeAsset->'%(RootDir)%(Directory)PDB/%(Filename)%(Extension)%(SymbolFileExtension)')"
+ Include="@(NativeRuntimeAsset->'%(RootDir)%(Directory)PDB/%(Filename)%(Extension)%(SymbolsSuffix)')"
IsSymbolFile="true"
IsNative="true" />
diff --git a/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Runtime.sfxproj b/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Runtime.sfxproj
index 75ef79bc75801..2402d7fbe2ef6 100644
--- a/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Runtime.sfxproj
+++ b/src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Runtime.sfxproj
@@ -44,8 +44,8 @@
-
-
+
+
@@ -108,10 +108,10 @@
- <_crossTargetJit Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == '$(LibraryFilePrefix)clrjit' and '%(Extension)' == '$(LibraryFileExtension)'" />
- <_clrjit Include="@(RuntimeFiles)" Condition="'%(FileName)' == '$(LibraryFilePrefix)clrjit' and '%(Extension)' == '$(LibraryFileExtension)'" />
- <_crossTargetCrossgen Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ApplicationFileExtension)'" />
- <_crossgen Include="@(RuntimeFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ApplicationFileExtension)'" />
+ <_crossTargetJit Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == '$(LibPrefix)clrjit' and '%(Extension)' == '$(LibSuffix)'" />
+ <_clrjit Include="@(RuntimeFiles)" Condition="'%(FileName)' == '$(LibPrefix)clrjit' and '%(Extension)' == '$(LibSuffix)'" />
+ <_crossTargetCrossgen Include="@(CoreCLRCrossTargetFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ExeSuffix)'" />
+ <_crossgen Include="@(RuntimeFiles)" Condition="'%(FileName)' == 'crossgen' and '%(Extension)' == '$(ExeSuffix)'" />
-
+
-
diff --git a/src/installer/tests/Directory.Build.targets b/src/installer/tests/Directory.Build.targets
index 6e2bc2f03f3d6..7691ad32cee76 100644
--- a/src/installer/tests/Directory.Build.targets
+++ b/src/installer/tests/Directory.Build.targets
@@ -106,7 +106,7 @@
- $(HostRuntimeIdentifier)
+ $(PackageRID)
$(MSBuildProjectName)
$(ArtifactsDir)tests/$(Configuration)/
diff --git a/src/libraries/Directory.Build.props b/src/libraries/Directory.Build.props
index 076d8145cc2b7..9d600a0efd951 100644
--- a/src/libraries/Directory.Build.props
+++ b/src/libraries/Directory.Build.props
@@ -1,6 +1,7 @@
true
+ true
@@ -19,24 +20,9 @@
$(RepositoryEngineeringDir)LicenseHeader.txt
-
- $(HostRuntimeIdentifier.Remove($(HostRuntimeIdentifier.LastIndexOf('-'))))
-
-
- $([System.Runtime.InteropServices.RuntimeInformation]::ProcessArchitecture.ToString().ToLowerInvariant)
- arm
- arm64
- wasm
- x64
- x64
-
-
- $(TargetOS.ToLowerInvariant())
-
Debug
$([System.Text.RegularExpressions.Regex]::Replace('$(TargetFramework)', '(-[^;]+)', ''))
@@ -93,57 +79,6 @@
-
- <_runtimeOSVersionIndex>$(RuntimeOS.IndexOfAny(".-0123456789"))
- <_runtimeOSFamily Condition="'$(_runtimeOSVersionIndex)' != '-1'">$(RuntimeOS.SubString(0, $(_runtimeOSVersionIndex)))
- <_portableOS>linux
- <_portableOS Condition="'$(RuntimeOS)' == 'linux-musl'">linux-musl
- <_portableOS Condition="'$(_runtimeOSFamily)' == 'win' or '$(TargetOS)' == 'windows'">win
- <_portableOS Condition="'$(_runtimeOSFamily)' == 'osx'">osx
- <_portableOS Condition="'$(_runtimeOSFamily)' == 'FreeBSD'">freebsd
- <_portableOS Condition="'$(_runtimeOSFamily)' == 'illumos'">illumos
- <_portableOS Condition="'$(_runtimeOSFamily)' == 'Solaris'">solaris
- <_portableOS Condition="'$(RuntimeOS)' == 'Browser'">browser
- <_portableOS Condition="'$(RuntimeOS)' == 'ios'">ios
- <_portableOS Condition="'$(RuntimeOS)' == 'tvos'">tvos
- <_portableOS Condition="'$(RuntimeOS)' == 'android'">android
-
- <_runtimeOS>$(RuntimeOS)
- <_runtimeOS Condition="'$(_runtimeOS)' == 'tizen.4.0.0'">linux
- <_runtimeOS Condition="'$(_runtimeOS)' == 'tizen.5.0.0'">linux
- <_runtimeOS Condition="'$(PortableBuild)' == 'true'">$(_portableOS)
- $(_runtimeOS)-x64
- $(_runtimeOS)-$(HostArch)
-
- linux-x64
-
-
- <_buildingInOSX>$([MSBuild]::IsOSPlatform('OSX'))
- win-x64
- osx-x64
- linux-x64
-
-
- win-x64
- osx-x64
- linux-x64
-
-
- osx-x64
- $(ToolRuntimeRID)
-
-
- <_portableOS Condition="'$(TargetOS)' == 'Unix' and '$(_runtimeOSFamily)' != 'osx' and '$(_runtimeOSFamily)' != 'FreeBSD' and '$(_runtimeOS)' != 'linux-musl' and '$(_runtimeOSFamily)' != 'illumos' and '$(_runtimeOSFamily)' != 'Solaris'">linux
-
-
- <_portableOS Condition="'$(TargetOS)' == 'Unix' and '$(_runtimeOSFamily)' != 'osx' and '$(_runtimeOSFamily)' != 'FreeBSD' and '$(_runtimeOS)' != 'linux-musl' and '$(_runtimeOSFamily)' != 'illumos' and '$(_runtimeOSFamily)' != 'Solaris'">linux
-
- <_packageRID />
- <_packageRID Condition="'$(PortableBuild)' == 'true'">$(_portableOS)-$(TargetArchitecture)
- $(_packageRID)
- $(RuntimeOS)-$(TargetArchitecture)
-
-
true
diff --git a/src/libraries/externals.csproj b/src/libraries/externals.csproj
index 20eb5e92624ef..1fece622cdb55 100644
--- a/src/libraries/externals.csproj
+++ b/src/libraries/externals.csproj
@@ -30,7 +30,7 @@
true
- false
+ false
-
- true
- true
- true
- true
- true
- true
- true
- true
- true
- true
- true
- true
- true
- true
-
-
8.0
diff --git a/src/tests/Common/ilasm/ilasm.ilproj b/src/tests/Common/ilasm/ilasm.ilproj
index 415d2ec53eaec..35c0b4ee0c519 100644
--- a/src/tests/Common/ilasm/ilasm.ilproj
+++ b/src/tests/Common/ilasm/ilasm.ilproj
@@ -4,6 +4,6 @@
needed by the IL SDK. -->
- $(TargetRid)
+ $(PackageRID)
diff --git a/src/tests/Common/publishdependency.targets b/src/tests/Common/publishdependency.targets
index 35ed438bc9962..596e4573756df 100644
--- a/src/tests/Common/publishdependency.targets
+++ b/src/tests/Common/publishdependency.targets
@@ -28,7 +28,7 @@
+ Properties="Language=C#;RuntimeIdentifier=$(PackageRID)" />
diff --git a/src/tests/Common/test_dependencies/test_dependencies.csproj b/src/tests/Common/test_dependencies/test_dependencies.csproj
index 106e50602676a..86c965ad15cba 100644
--- a/src/tests/Common/test_dependencies/test_dependencies.csproj
+++ b/src/tests/Common/test_dependencies/test_dependencies.csproj
@@ -5,7 +5,7 @@
$(NetCoreAppToolCurrent)
true
true
- win-arm;win-arm64;win-x64;win-x86;$(TargetRid)
+ win-arm;win-arm64;win-x64;win-x86;$(PackageRID)
true
Release
diff --git a/src/tests/Common/test_dependencies_fs/test_dependencies.fsproj b/src/tests/Common/test_dependencies_fs/test_dependencies.fsproj
index ab7b191f39c39..6a544c39f9837 100644
--- a/src/tests/Common/test_dependencies_fs/test_dependencies.fsproj
+++ b/src/tests/Common/test_dependencies_fs/test_dependencies.fsproj
@@ -5,7 +5,7 @@
$(NetCoreAppToolCurrent)
true
true
- win-arm;win-arm64;win-x64;win-x86;$(TargetRid)
+ win-arm;win-arm64;win-x64;win-x86;$(PackageRID)
true
Release
diff --git a/src/tests/Directory.Build.props b/src/tests/Directory.Build.props
index 7b834896308bc..c0d820eb7de80 100644
--- a/src/tests/Directory.Build.props
+++ b/src/tests/Directory.Build.props
@@ -106,65 +106,6 @@
false
-
-
-
-
-
-
-
-
-
- true
- win-$(TargetArchitecture)
-
-
-
-
- true
- true
- ubuntu.14.04-$(TargetArchitecture)
-
-
-
-
- true
- true
- osx.10.12-$(TargetArchitecture)
-
-
-
-
- true
- true
- ubuntu.14.04-$(TargetArchitecture)
-
-
-
-
- true
- true
- ubuntu.14.04-$(TargetArchitecture)
-
-
-
-
- true
- true
- ubuntu.14.04-$(TargetArchitecture)
-
-
-
-
- true
- true
- ubuntu.14.04-$(TargetArchitecture)
-
-
-
-
32
@@ -172,21 +113,6 @@
64
-
- $(__RuntimeId)
- $(TestNugetRuntimeId)
-
-
-
-
- $(TargetRid)
- $(TargetRid)
-
-
true
C#
diff --git a/src/tests/build.cmd b/src/tests/build.cmd
index 1003f48063404..e3bc02a54df5f 100644
--- a/src/tests/build.cmd
+++ b/src/tests/build.cmd
@@ -52,7 +52,6 @@ set __SkipManaged=
set __SkipTestWrappers=
set __BuildTestWrappersOnly=
set __SkipNative=
-set __RuntimeId=
set __TargetsWindows=1
set __DoCrossgen=
set __DoCrossgen2=
@@ -109,7 +108,6 @@ if /i "%1" == "buildagainstpackages" (echo error: Remove /BuildAgainstPackages
if /i "%1" == "crossgen" (set __DoCrossgen=1&set __TestBuildMode=crossgen&set processedArgs=!processedArgs! %1&shift&goto Arg_Loop)
if /i "%1" == "crossgen2" (set __DoCrossgen2=1&set __TestBuildMode=crossgen2&set processedArgs=!processedArgs! %1&shift&goto Arg_Loop)
if /i "%1" == "composite" (set __CompositeBuildMode=1&set __DoCrossgen2=1&set __TestBuildMode=crossgen2&set processedArgs=!processedArgs! %1&shift&goto Arg_Loop)
-if /i "%1" == "runtimeid" (set __RuntimeId=%2&set processedArgs=!processedArgs! %1 %2&shift&shift&goto Arg_Loop)
if /i "%1" == "targetsNonWindows" (set __TargetsWindows=0&set processedArgs=!processedArgs! %1&shift&goto Arg_Loop)
if /i "%1" == "Exclude" (set __Exclude=%2&set processedArgs=!processedArgs! %1 %2&shift&shift&goto Arg_Loop)
if /i "%1" == "-priority" (set __Priority=%2&shift&set processedArgs=!processedArgs! %1=%2&shift&goto Arg_Loop)
@@ -426,11 +424,6 @@ REM ============================================================================
echo %__MsgPrefix%Creating test overlay
-set RuntimeIdArg=
-if defined __RuntimeId (
- set RuntimeIdArg=/p:RuntimeId="%__RuntimeId%"
-)
-
set __BuildLogRootName=Tests_Overlay_Managed
set __BuildLog=%__LogsDir%\%__BuildLogRootName%_%__TargetOS%__%__BuildArch%__%__BuildType%.log
set __BuildWrn=%__LogsDir%\%__BuildLogRootName%_%__TargetOS%__%__BuildArch%__%__BuildType%.wrn
@@ -445,7 +438,7 @@ powershell -NoProfile -ExecutionPolicy ByPass -NoLogo -File "%__RepoRootDir%\eng
/p:RestoreDefaultOptimizationDataPackage=false /p:PortableBuild=true^
/p:UsePartialNGENOptimization=false /maxcpucount^
%__SkipFXRestoreArg%^
- !__Logging! %__CommonMSBuildArgs% %RuntimeIdArg% %__PriorityArg% %__BuildNeedTargetArg% %__UnprocessedBuildArgs%
+ !__Logging! %__CommonMSBuildArgs% %__PriorityArg% %__BuildNeedTargetArg% %__UnprocessedBuildArgs%
if errorlevel 1 (
echo %__ErrMsgPrefix%%__MsgPrefix%Error: Create Test Overlay failed. Refer to the build log files for details:
echo %__BuildLog%
@@ -560,7 +553,6 @@ echo Build type: one of Debug, Checked, Release ^(default: Debug^).
echo skipmanaged: skip the managed tests build
echo skipnative: skip the native tests build
echo skiprestorepackages: skip package restore
-echo runtimeid ^: Builds a test overlay for the specified OS ^(Only supported when building against packages^). Supported IDs are:
echo alpine.3.4.3-x64: Builds overlay for Alpine 3.4.3
echo debian.8-x64: Builds overlay for Debian 8
echo fedora.24-x64: Builds overlay for Fedora 24
diff --git a/src/tests/run.proj b/src/tests/run.proj
index 15e44d1a6f68a..1841c69bc5e63 100644
--- a/src/tests/run.proj
+++ b/src/tests/run.proj
@@ -445,22 +445,22 @@ namespace $([System.String]::Copy($(Category)).Replace(".","_").Replace("\","").
-
-
-
-
-
+
+ Properties="Language=C#;PackageRID=$(PackageRID);RuntimeIdentifier=$(PackageRID)" />
From ee9471afbdc8b8fc8804439e6c1f4efe5ef000e7 Mon Sep 17 00:00:00 2001
From: Adeel <3840695+am11@users.noreply.github.com>
Date: Thu, 19 Nov 2020 03:30:31 +0200
Subject: [PATCH 2/3] Delete legacy tooling properties
---
eng/Configurations.props | 1 -
src/installer/Directory.Build.props | 21 -----------
src/installer/Directory.Build.targets | 54 ---------------------------
src/tests/build.cmd | 13 -------
4 files changed, 89 deletions(-)
diff --git a/eng/Configurations.props b/eng/Configurations.props
index 6f68f208fc0f5..c4bd905ddc627 100644
--- a/eng/Configurations.props
+++ b/eng/Configurations.props
@@ -100,7 +100,6 @@
<_portableOS Condition="'$(_runtimeOS)' == 'android'">android
<_runtimeOS Condition="$(_runtimeOS.StartsWith('tizen'))">linux
- $(_runtimeOS)
<_runtimeOS Condition="'$(PortableBuild)' == 'true'">$(_portableOS)
diff --git a/src/installer/Directory.Build.props b/src/installer/Directory.Build.props
index e7bbc71a7a739..964dfec670285 100644
--- a/src/installer/Directory.Build.props
+++ b/src/installer/Directory.Build.props
@@ -112,27 +112,6 @@
$(SharedHostInstallerStart)$(InstallerStartSuffix)-
$(HostFxrInstallerStart)$(InstallerStartSuffix)-
$(SharedFrameworkInstallerStart)$(InstallerStartSuffix)-
-
-
-
-
- true
-
- true
-
-
-
- .zip
- .tar.gz
- .msi
- .pkg
- .deb
- .rpm
- .exe
- $(InstallerExtension)
diff --git a/src/installer/Directory.Build.targets b/src/installer/Directory.Build.targets
index 7a859d808b229..f513c96c1e30b 100644
--- a/src/installer/Directory.Build.targets
+++ b/src/installer/Directory.Build.targets
@@ -142,60 +142,6 @@
-
-
- $(CombinedInstallerStart)$(ProductMoniker)$(CombinedInstallerExtension)
- $(CombinedInstallerStart)$(ProductMoniker)-engine.exe
-
- $(SharedHostInstallerStart)$(ProductMoniker)$(InstallerExtension)
- $(HostFxrInstallerStart)$(HostResolverVersionMoniker)$(InstallerExtension)
- $(SharedFrameworkInstallerStart)$(ProductMoniker)$(InstallerExtension)
- $(DotnetRuntimeDependenciesPackageInstallerStart)$(ProductMoniker)$(InstallerExtension)
-
- dotnet-runtime-$(ProductMoniker)$(CompressedFileExtension)
- dotnet-hostfxr-internal-$(PackageTargetRid).$(HostResolverVersion)$(CompressedFileExtension)
- dotnet-nethost-$(AppHostVersion)-$(PackageTargetRid)$(CompressedFileExtension)
- dotnet-crossgen2-$(ProductMoniker)$(CompressedFileExtension)
- dotnet-runtime-internal-$(ProductMoniker)$(CompressedFileExtension)
- dotnet-runtime-symbols-$(ProductMoniker)$(CompressedFileExtension)
-
- $(DotnetHostFxrString)$(ProductBandVersion)
- $(HostFxrDebPkgName.ToLower())
- $(DotnetRuntimeString)$(ProductBandVersion)
- $(SharedFxDebPkgName.ToLower())
- $(DotnetRuntimeDependenciesPackageString)$(ProductBandVersion)
- $(RuntimeDependenciesDebPkgName.ToLower())
-
- dotnet-hostfxr-$(ProductBandVersion)
- $(HostFxrRpmPkgName.ToLower())
- dotnet-runtime-$(ProductBandVersion)
- $(SharedFxRpmPkgName.ToLower())
- $(DotnetRuntimeDependenciesPackageString)$(ProductBandVersion)
- $(RuntimeDependenciesRpmPkgName.ToLower())
-
-
-
- $(AssetOutputPath)dotnet-targeting-pack-$(ProductMoniker)$(InstallerExtension)
- $(AssetOutputPath)dotnet-apphost-pack-$(ProductMoniker)$(InstallerExtension)
- 2.1
- $(NetStandardProductBandVersion).$(PatchVersion)$(ProductVersionSuffix)-$(PackageTargetRid)
- $(AssetOutputPath)netstandard-targeting-pack-$(NetStandardProductMoniker)$(InstallerExtension)
-
-
-
-
- $(SharedHostInstallerStart)$(SharedFrameworkNugetVersion)-$(TargetArchitecture)$(InstallerExtension)
- $(HostFxrInstallerStart)$(HostResolverVersion)-$(TargetArchitecture)$(InstallerExtension)
- $(SharedFrameworkInstallerStart)$(SharedFrameworkNugetVersion)-$(TargetArchitecture)$(InstallerExtension)
-
-
-
-
- $(DotnetRuntimeDependenciesPackageInstallerStart)$(SharedFrameworkNugetVersion)-$(TargetArchitecture)$(InstallerExtension)
-
-
-
diff --git a/src/tests/build.cmd b/src/tests/build.cmd
index e3bc02a54df5f..3dca37ac6543b 100644
--- a/src/tests/build.cmd
+++ b/src/tests/build.cmd
@@ -553,19 +553,6 @@ echo Build type: one of Debug, Checked, Release ^(default: Debug^).
echo skipmanaged: skip the managed tests build
echo skipnative: skip the native tests build
echo skiprestorepackages: skip package restore
-echo alpine.3.4.3-x64: Builds overlay for Alpine 3.4.3
-echo debian.8-x64: Builds overlay for Debian 8
-echo fedora.24-x64: Builds overlay for Fedora 24
-echo linux-x64: Builds overlay for portable linux
-echo opensuse.42.1-x64: Builds overlay for OpenSUSE 42.1
-echo osx.10.12-x64: Builds overlay for OSX 10.12
-echo osx-x64: Builds overlay for portable OSX
-echo rhel.7-x64: Builds overlay for RHEL 7 or CentOS
-echo ubuntu.14.04-x64: Builds overlay for Ubuntu 14.04
-echo ubuntu.16.04-x64: Builds overlay for Ubuntu 16.04
-echo ubuntu.16.10-x64: Builds overlay for Ubuntu 16.10
-echo win-x64: Builds overlay for portable Windows
-echo win7-x64: Builds overlay for Windows 7
echo crossgen: Precompiles the framework managed assemblies
echo copynativeonly: Only copy the native test binaries to the managed output. Do not build the native or managed tests.
echo skipgeneratelayout: Do not generate the Core_Root layout
From 311fabeb168aa7de926e85b761da2c1b31461b6c Mon Sep 17 00:00:00 2001
From: Adeel <3840695+am11@users.noreply.github.com>
Date: Thu, 19 Nov 2020 07:10:58 +0200
Subject: [PATCH 3/3] Delete legacy tooling properties
---
src/installer/Directory.Build.props | 15 ---------------
src/installer/Directory.Build.targets | 5 -----
2 files changed, 20 deletions(-)
diff --git a/src/installer/Directory.Build.props b/src/installer/Directory.Build.props
index 964dfec670285..15472af36dcb9 100644
--- a/src/installer/Directory.Build.props
+++ b/src/installer/Directory.Build.props
@@ -114,13 +114,6 @@
$(SharedFrameworkInstallerStart)$(InstallerStartSuffix)-
-
-
- $(OutputRid)
- osx.10.10-x64
- rhel.7-x64
-
-
true
-
-
- .map
- .ni.pdb
-
-
-
-
diff --git a/src/installer/Directory.Build.targets b/src/installer/Directory.Build.targets
index f513c96c1e30b..dc2e7c7eed40c 100644
--- a/src/installer/Directory.Build.targets
+++ b/src/installer/Directory.Build.targets
@@ -89,11 +89,6 @@
Runtime/$(SharedFrameworkNugetVersion)/
-
- $(SharedFrameworkNugetVersion)-$(PackageTargetRid)
- $(HostResolverVersion)-$(PackageTargetRid)
-
-
$(HostVersion)
1