Skip to content

Commit fe24e02

Browse files
committed
Change how the SDK looks things up as it publishes blobs in non-standard ways (and it was picking up installers in the linux PGO legs)
1 parent dbb743e commit fe24e02

File tree

1 file changed

+44
-31
lines changed

1 file changed

+44
-31
lines changed

eng/Publishing.props

Lines changed: 44 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -151,37 +151,50 @@
151151
<InstallersAssetManifestFilePath>$(ArtifactsLogDir)AssetManifest\$(InstallersAssetManifestFileName).xml</InstallersAssetManifestFilePath>
152152
</PropertyGroup>
153153

154-
<ItemGroup>
155-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.zip" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
156-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.tar.gz" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
157-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.pkg" />
158-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.exe" />
159-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.deb" />
160-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.rpm" />
161-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.msi" />
162-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.cab" />
163-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.svg" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
164-
<!-- Only publish this file from windows x64 so that we don't end up with duplicates -->
165-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)productVersion.txt"
166-
Condition=" '$(PublishBinariesAndBadge)' == 'true' and '$(OS)' == 'Windows_NT' and '$(Architecture)' == 'x64' and '$(PgoInstrument)' != 'true'" />
167-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)sdk-productVersion.txt"
168-
Condition=" '$(PublishBinariesAndBadge)' == 'true' and '$(OS)' == 'Windows_NT' and '$(Architecture)' == 'x64' and '$(PgoInstrument)' != 'true'" />
169-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)productCommit-*.json" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
170-
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)productCommit-*.txt" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
171-
<SdkNonShippingAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)*.swr" />
172-
<SdkNonShippingAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)*.msi" />
173-
<SdkNonShippingAssetsToPublish Condition="'$(PublishBinariesAndBadge)' != 'false'" Include="$(ArtifactsNonShippingPackagesDir)*.tar.gz" />
174-
<!-- dotnet-toolset zips are published in the PublishToolsetAssets target. -->
175-
<SdkNonShippingAssetsToPublish Condition="'$(PublishBinariesAndBadge)' != 'false'" Include="$(ArtifactsNonShippingPackagesDir)*.zip" Exclude="$(ArtifactsNonShippingPackagesDir)dotnet-toolset*.zip" />
176-
<SdkNonShippingAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)*.pkg" />
177-
<CheckSumsToPublish Include="$(ArtifactsShippingPackagesDir)*.sha512" />
178-
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)productCommit-*.json.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
179-
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)productCommit-*.txt.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
180-
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)productVersion.txt.sha512" Condition=" '$(OS)' != 'Windows_NT' or '$(Architecture)' != 'x64' " />
181-
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)sdk-productVersion.txt.sha512" Condition=" '$(OS)' != 'Windows_NT' or '$(Architecture)' != 'x64' " />
182-
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)*.zip.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
183-
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)*.tar.gz.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
184-
</ItemGroup>
154+
<!-- When doing a PGO build in the VMR, only ship the SDK's PGO archives -->
155+
<Choose>
156+
<When Condition="'$(PgoInstrument)' == 'true' and '$(DotNetBuildOrchestrator)' == 'true'">
157+
<ItemGroup>
158+
<SdkAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)/dotnet-sdk-pgo-*.zip" />
159+
<SdkAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)/dotnet-sdk-pgo-*.tar.gz" />
160+
<CheckSumsToPublish Include="$(ArtifactsNonShippingPackagesDir)/dotnet-sdk-pgo-*.zip.sha512" />
161+
<CheckSumsToPublish Include="$(ArtifactsNonShippingPackagesDir)/dotnet-sdk-pgo-*.tar.gz.sha512" />
162+
</ItemGroup>
163+
</When>
164+
<Otherwise>
165+
<ItemGroup>
166+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.zip" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
167+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.tar.gz" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
168+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.pkg" />
169+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.exe" />
170+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.deb" />
171+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.rpm" />
172+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.msi" />
173+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.cab" />
174+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)*.svg" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
175+
<!-- Only publish this file from windows x64 so that we don't end up with duplicates -->
176+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)productVersion.txt"
177+
Condition=" '$(PublishBinariesAndBadge)' == 'true' and '$(OS)' == 'Windows_NT' and '$(Architecture)' == 'x64' and '$(PgoInstrument)' != 'true'" />
178+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)sdk-productVersion.txt"
179+
Condition=" '$(PublishBinariesAndBadge)' == 'true' and '$(OS)' == 'Windows_NT' and '$(Architecture)' == 'x64' and '$(PgoInstrument)' != 'true'" />
180+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)productCommit-*.json" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
181+
<SdkAssetsToPublish Include="$(ArtifactsShippingPackagesDir)productCommit-*.txt" Condition=" '$(PublishBinariesAndBadge)' == 'true' " />
182+
<SdkNonShippingAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)*.swr" />
183+
<SdkNonShippingAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)*.msi" />
184+
<SdkNonShippingAssetsToPublish Condition="'$(PublishBinariesAndBadge)' != 'false'" Include="$(ArtifactsNonShippingPackagesDir)*.tar.gz" />
185+
<!-- dotnet-toolset zips are published in the PublishToolsetAssets target. -->
186+
<SdkNonShippingAssetsToPublish Condition="'$(PublishBinariesAndBadge)' != 'false'" Include="$(ArtifactsNonShippingPackagesDir)*.zip" Exclude="$(ArtifactsNonShippingPackagesDir)dotnet-toolset*.zip" />
187+
<SdkNonShippingAssetsToPublish Include="$(ArtifactsNonShippingPackagesDir)*.pkg" />
188+
<CheckSumsToPublish Include="$(ArtifactsShippingPackagesDir)*.sha512" />
189+
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)productCommit-*.json.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
190+
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)productCommit-*.txt.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
191+
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)productVersion.txt.sha512" Condition=" '$(OS)' != 'Windows_NT' or '$(Architecture)' != 'x64' " />
192+
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)sdk-productVersion.txt.sha512" Condition=" '$(OS)' != 'Windows_NT' or '$(Architecture)' != 'x64' " />
193+
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)*.zip.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
194+
<CheckSumsToPublish Remove="$(ArtifactsShippingPackagesDir)*.tar.gz.sha512" Condition=" '$(PublishBinariesAndBadge)' == 'false' " />
195+
</ItemGroup>
196+
</Otherwise>
197+
</Choose>
185198

186199
<!-- Publish the sdk and the checksums only in the initial build pass. -->
187200
<Target Name="PublishSdkAssetsAndChecksums"

0 commit comments

Comments
 (0)