Skip to content

Commit 6ff57f1

Browse files
authored
Actually use TargetOS=AnyOS in the build-test-job. (dotnet#61745)
1 parent c0dabb5 commit 6ff57f1

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

eng/pipelines/common/templates/runtimes/build-test-job.yml

+6-2
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,13 @@ jobs:
8989
- ${{ if eq(parameters.runtimeFlavor, 'coreclr') }}:
9090
- name: liveRuntimeBuildParams
9191
value: ${{ format('clr.corelib+libs.ref+libs.native -rc {0} -c {1} -arch {2} -ci', coalesce(parameters.liveRuntimeBuildConfig, parameters.buildConfig), parameters.liveLibrariesBuildConfig, parameters.archType) }}
92+
- name: liveRuntimeArtifactsPathArg
93+
value: ${{ format('/p:CoreCLROverridePath={0}/artifacts/bin/coreclr/{1}{2}.{3}.{4}', $(Build.SourcesDirectory), parameters.osGroup, parameters.osSubgroup, parameters.archType, coalesce(parameters.liveRuntimeBuildConfig, parameters.buildConfig)) }}
9294
- ${{ if eq(parameters.runtimeFlavor, 'mono') }}:
9395
- name: liveRuntimeBuildParams
9496
value: ${{ format('mono.corelib+libs.ref+libs.native -rc {0} -c {1} -arch {2} -ci', coalesce(parameters.liveRuntimeBuildConfig, parameters.buildConfig), parameters.liveLibrariesBuildConfig, parameters.archType) }}
97+
- name: liveRuntimeArtifactsPathArg
98+
value: ${{ format('/p:MonoOverridePath={0}/artifacts/bin/mono/{1}{2}.{3}.{4}', $(Build.SourcesDirectory), parameters.osGroup, parameters.osSubgroup, parameters.archType, coalesce(parameters.liveRuntimeBuildConfig, parameters.buildConfig)) }}
9599
- name: compilerArg
96100
value: ''
97101
- ${{ if and(ne(parameters.osGroup, 'windows'), ne(parameters.compilerName, 'gcc')) }}:
@@ -135,7 +139,7 @@ jobs:
135139
displayName: Disk Usage before Build
136140
137141
# Build managed test components
138-
- script: $(Build.SourcesDirectory)/src/tests/build$(scriptExt) $(logRootNameArg)Managed allTargets skipnative skipgeneratelayout skiptestwrappers $(buildConfig) $(archType) $(runtimeFlavorArgs) $(crossArg) $(priorityArg) ci $(librariesOverrideArg)
142+
- script: $(Build.SourcesDirectory)/src/tests/build$(scriptExt) $(logRootNameArg)Managed allTargets skipnative skipgeneratelayout skiptestwrappers $(buildConfig) $(archType) $(runtimeFlavorArgs) $(crossArg) $(priorityArg) ci $(librariesOverrideArg) /p:RuntimeOS=${{ parameters.osGroup }}${{ parameters.osSubgroup }} /p:LibrariesTargetOSConfigurationArchitecture=${{ parameters.osGroup }}${{ parameters.osSubgroup }}-${{ parameters.liveLibrariesBuildConfig }}-${{ parameters.archType }} $(liveRuntimeArtifactsPathArg) /p:TargetOS=AnyOS
139143
displayName: Build managed test components
140144

141145
- ${{ if in(parameters.osGroup, 'OSX', 'iOS', 'tvOS') }}:
@@ -147,7 +151,7 @@ jobs:
147151
# Zip and publish managed test components
148152
- template: /eng/pipelines/common/upload-artifact-step.yml
149153
parameters:
150-
rootFolder: $(managedTestArtifactRootFolderPath)
154+
rootFolder: '$(binTestsPath)/AnyOS.$(archType).$(buildConfigUpper)'
151155
includeRootFolder: false
152156
archiveExtension: '.tar.gz'
153157
archiveType: tar

src/tests/build.proj

+3-2
Original file line numberDiff line numberDiff line change
@@ -394,8 +394,6 @@
394394

395395
<Target Name="BuildTargetingPack" AfterTargets="BatchRestorePackages">
396396
<Message Text="$(MsgPrefix)Building Targeting Pack" Importance="High" />
397-
<Error Text="$(ErrMsgPrefix)$(MsgPrefix)ERROR: TargetOS has not been specified. Please do that then run build again."
398-
Condition="'$(TargetOS)' == 'AnyOS'" />
399397
<MSBuild Projects="Common\external\external.csproj"
400398
Targets="Build" />
401399
</Target>
@@ -450,14 +448,17 @@
450448
<GroupBuildCmd>$(GroupBuildCmd) "/p:TargetArchitecture=$(TargetArchitecture)"</GroupBuildCmd>
451449
<GroupBuildCmd>$(GroupBuildCmd) "/p:Configuration=$(Configuration)"</GroupBuildCmd>
452450
<GroupBuildCmd>$(GroupBuildCmd) "/p:LibrariesConfiguration=$(LibrariesConfiguration)"</GroupBuildCmd>
451+
<GroupBuildCmd>$(GroupBuildCmd) "/p:LibrariesTargetOSConfigurationArchitecture=$(LibrariesTargetOSConfigurationArchitecture)"</GroupBuildCmd>
453452
<GroupBuildCmd>$(GroupBuildCmd) "/p:TargetOS=$(TargetOS)"</GroupBuildCmd>
453+
<GroupBuildCmd>$(GroupBuildCmd) "/p:RuntimeOS=$(RuntimeOS)"</GroupBuildCmd>
454454
<GroupBuildCmd>$(GroupBuildCmd) "/p:RuntimeFlavor=$(RuntimeFlavor)"</GroupBuildCmd>
455455
<GroupBuildCmd>$(GroupBuildCmd) "/p:RuntimeVariant=$(RuntimeVariant)"</GroupBuildCmd>
456456
<GroupBuildCmd>$(GroupBuildCmd) "/p:CLRTestBuildAllTargets=$(CLRTestBuildAllTargets)"</GroupBuildCmd>
457457
<GroupBuildCmd>$(GroupBuildCmd) "/p:__TestGroupToBuild=$(__TestGroupToBuild)"</GroupBuildCmd>
458458
<GroupBuildCmd>$(GroupBuildCmd) "/p:__SkipRestorePackages=1"</GroupBuildCmd>
459459
<GroupBuildCmd>$(GroupBuildCmd) /nodeReuse:false</GroupBuildCmd>
460460
<GroupBuildCmd>$(GroupBuildCmd) /maxcpucount</GroupBuildCmd>
461+
<GroupBuildCmd>$(GroupBuildCmd) /bl:$(ArtifactsDir)/log/$(Configuration)/InnerManagedTestBuild.$(__TestGroupToBuild).binlog</GroupBuildCmd>
461462
</PropertyGroup>
462463

463464
<Message Importance="High" Text="$(MsgPrefix)Building managed test group $(__TestGroupToBuild): $(GroupBuildCmd)" />

0 commit comments

Comments
 (0)