Skip to content

Commit ff77788

Browse files
authored
Get rid of IsPublicRuntime, use AdditionalDotNetPackageFeed (#44605)
1 parent 284c84e commit ff77788

File tree

1 file changed

+6
-22
lines changed

1 file changed

+6
-22
lines changed

eng/helix/helix.proj

Lines changed: 6 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,4 @@
11
<Project Sdk="Microsoft.DotNet.Helix.Sdk" DefaultTargets="Test" TreatAsLocalProperty="ProjectToBuild">
2-
3-
<!--
4-
TEMPORARY workaround while we wait for https://github.com/dotnet/arcade/issues/8336 to be addressed.
5-
When the runtime version we're ingesting was built publicly, or we're building in public CI, IsPublicRuntime should be true,
6-
causing us to use the normal workflow of downloading it from the public location in Helix.
7-
When the runtime was built internally (which necessarily also means we're building in the internal project), IsPublicRuntime should be false,
8-
causing us to pass the right Sas token to get Helx to download it from the internal location.
9-
Once https://github.com/dotnet/arcade/issues/8336, we only need to set AdditionalDotNetPackage below,
10-
not HelixCorrelationPayload.
11-
-->
12-
<PropertyGroup>
13-
<IsPublicRuntime>false</IsPublicRuntime>
14-
</PropertyGroup>
15-
162
<PropertyGroup>
173
<!--
184
When invoking helix.proj for the whole repo with build.cmd, ProjectToBuild will be set to the path to this project.
@@ -60,17 +46,15 @@
6046
<ItemGroup>
6147
<!--
6248
Use the BrowserDebugHost transport package as a sentinel for the non-shipping version of the NETCoreApp shared framework.
63-
!temporary! e.g. https://dotnetbuilds.blob.core.windows.net/internal/Runtime/6.0.2-servicing.22060.12/dotnet-runtime-6.0.2-win-x86.zip?{token}
6449
-->
65-
<HelixCorrelationPayload Include="dotnet-cli"
66-
Condition="'$(SYSTEM_TEAMPROJECT)' == 'internal' AND '$(IsPublicRuntime)' == 'false'"
67-
Destination="dotnet-cli"
68-
Uri="https://dotnetbuilds.blob.core.windows.net/internal/Runtime/$(MicrosoftNETCoreBrowserDebugHostTransportVersion)/dotnet-runtime-$(MicrosoftNETCoreAppRuntimeVersion)-$(DotNetCliRuntime)$(ArchiveExtension)$([System.Environment]::GetEnvironmentVariable('DotNetBuildsInternalReadSasToken'))" />
69-
70-
<AdditionalDotNetPackage Include="$(MicrosoftNETCoreBrowserDebugHostTransportVersion)"
71-
Condition="'$(SYSTEM_TEAMPROJECT)' != 'internal' OR '$(IsPublicRuntime)' == 'true'">
50+
<AdditionalDotNetPackage Include="$(MicrosoftNETCoreBrowserDebugHostTransportVersion)">
7251
<PackageType>runtime</PackageType>
7352
</AdditionalDotNetPackage>
53+
54+
<AdditionalDotNetPackageFeed Include="https://dotnetbuilds.blob.core.windows.net/internal"
55+
Condition="'$(SYSTEM_TEAMPROJECT)' == 'internal'">
56+
<SasToken>$([System.Environment]::GetEnvironmentVariable('DotNetBuildsInternalReadSasToken'))</SasToken>
57+
</AdditionalDotNetPackageFeed>
7458
</ItemGroup>
7559

7660
<PropertyGroup Condition=" '$(ContinuousIntegrationBuild)' == 'true' ">

0 commit comments

Comments
 (0)