-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Conversation
After dotnet#26108, this was broken because the Tools.proj was never restored on the agent running the finalize-publish job. This fixes the script to do a restore, and adds a missing import.
UpdatePublishedVersions.ps1
Outdated
/p:UsePartialNGENOptimization=false ` | ||
/maxcpucount ` | ||
"$PSScriptRoot\eng\updateversions.proj" /t:UpdatePublishedVersions ` | ||
& "$PSScriptRoot\eng\common\build.ps1" -restore -build ` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this script doesn't have to do build tools initialization now, is there a reason for us to still have a separate script instead of just having the step in yml call build.ps1 directly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also pass -ci? Considering this is mean only for use in the pipeline?
eng/updateversions.proj
Outdated
@@ -6,6 +6,10 @@ | |||
|
|||
<Import Project="$([MSBuild]::GetPathOfFileAbove('Directory.Build.props', '$(MSBuildThisFileDirectory)../'))" /> | |||
|
|||
<Import Project="$(NuGetPackageRoot)\microsoft.dotnet.versiontools.tasks\$(MicrosoftDotNetVersionToolsTasksPackageVersion)\build\Microsoft.DotNet.VersionTools.Tasks.props" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need to add Microsoft.DotNet.VersionTools.Tasks
as a repo dependency for auto-updates? (I don't see it in Version.Details.xml
)
UpdatePublishedVersions.ps1
Outdated
/p:UsePartialNGENOptimization=false ` | ||
/maxcpucount ` | ||
"$PSScriptRoot\eng\updateversions.proj" /t:UpdatePublishedVersions ` | ||
& "$PSScriptRoot\eng\common\build.ps1" -restore -build ` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also pass -ci? Considering this is mean only for use in the pipeline?
After #26108, this was broken
because the Tools.proj was never restored on the agent running the
finalize-publish job. This fixes the script to do a restore, and adds
a missing import.
@dotnet/coreclr-infra PTAL