-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Closed
Description
We should switch to the well defined RID system away from our current defined OS configurations. This allows us to share the OS configuration matrix with installer and have just one central point where this is defined.
See
runtime/src/installer/Directory.Build.props
Lines 227 to 308 in 9704493
<Choose> | |
<When Condition="$(OutputRid.StartsWith('win'))"> | |
<PropertyGroup> | |
<TargetsWindows>true</TargetsWindows> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('osx'))"> | |
<PropertyGroup> | |
<TargetsOSX>true</TargetsOSX> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('debian'))"> | |
<PropertyGroup> | |
<TargetsDebian>true</TargetsDebian> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('ubuntu'))"> | |
<PropertyGroup> | |
<TargetsUbuntu>true</TargetsUbuntu> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('linuxmint'))"> | |
<PropertyGroup> | |
<TargetsLinuxMint>true</TargetsLinuxMint> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('rhel'))"> | |
<PropertyGroup> | |
<TargetsRhel>true</TargetsRhel> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('centos'))"> | |
<PropertyGroup> | |
<TargetsCentos>true</TargetsCentos> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('opensuse'))"> | |
<PropertyGroup> | |
<TargetsOpensuse>true</TargetsOpensuse> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('fedora'))"> | |
<PropertyGroup> | |
<TargetsFedora>true</TargetsFedora> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('oracle'))"> | |
<PropertyGroup> | |
<TargetsOracle>true</TargetsOracle> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<When Condition="$(OutputRid.StartsWith('sles'))"> | |
<PropertyGroup> | |
<TargetsSles>true</TargetsSles> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</When> | |
<Otherwise> | |
<PropertyGroup> | |
<TargetsLinux>true</TargetsLinux> | |
<TargetsUnix>true</TargetsUnix> | |
</PropertyGroup> | |
</Otherwise> | |
</Choose> |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Metadata
Assignees
Labels
area-Infrastructure-librarieshelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributors