Bump Proc to 0.14.0, retry dotnet tool restore once on failure - #31
Merged
Merged
Conversation
Proc was pinned at 0.9.1; catches it up to 0.14.0 to match the semantics (no shell expansion, throws on failure) the exec helper already assumes. Also retries `dotnet tool restore` once on failure: dotnet/sdk#53783 causes cold-cache restores of 2+ RID-specific tool packages (nupkg-validator and assembly-differ, both native-AOT now) to misattribute one package's DotnetToolSettings.xml to another. A retry always succeeds since the resolver cache is warm afterward. Co-authored-by: Cursor <cursoragent@cursor.com>
The plain "release-notes" package id is a DotnetToolSettings.xml v2 shim that maps RIDs to per-RID sub-packages (see generatePackages) - it ships no managed assembly of its own. Diffing against it found 0 assemblies and failed the PR's own "Inspect public API changes" CI step: https://github.com/nullean/release-notes/actions/runs/33674817669/job/100397363448 The portable, signed managed build lives in "release-notes.any" instead; point previous-nuget at that. Verified locally against the real 0.11.0 package. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
Author
|
Also fixed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Procbuild-script dependency from 0.9.1 to 0.14.0 (latest), matching the semantics (no shell expansion, throws on failure) already assumed elsewhere.dotnet tool restorenow retries once on failure: dotnet/sdk#53783 causes cold-cache restores of 2+ RID-specific tool packages (nupkg-validatorandassembly-differ, both native-AOT now) to misattribute one package'sDotnetToolSettings.xmlto another. A retry always succeeds since the cache is warm afterward.Test plan
dotnet build build/scripts/scripts.fsproj -c Releasesucceeds.Made with Cursor