Skip to content

Restore src/tests packages in one-shot#126405

Merged
jkoritzinsky merged 3 commits intodotnet:mainfrom
am11:patch-51
Apr 1, 2026
Merged

Restore src/tests packages in one-shot#126405
jkoritzinsky merged 3 commits intodotnet:mainfrom
am11:patch-51

Conversation

@am11
Copy link
Copy Markdown
Member

@am11 am11 commented Apr 1, 2026

On rebuild of src/tests/build.cmd, before:

    [10:26:11.23] Restoring all packages...

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    Restored C:\temp\runtime\src\tests\Common\XUnitLogChecker\XUnitLogChecker.csproj (in 171 ms).

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.

    Determining projects to restore...
    All projects are up-to-date for restore.
  [10:29:49.75] Restoring all packages...Done.

and after:

  [14:38:42.29] Restoring all packages...
  Determining projects to restore...
  All projects are up-to-date for restore.
  [14:38:43.48] Restoring all packages...Done.

Fixes #123633

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Apr 1, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@am11
Copy link
Copy Markdown
Member Author

am11 commented Apr 1, 2026

@jkoritzinsky, I can add a second pass for ForceRestore (which I've deleted), but I'm not sure if it's needed. crossgen2, cdac etc. are on the same plan in product build (restores for both host and target alike) and it just works. Could you confirm?

@jkoritzinsky
Copy link
Copy Markdown
Member

I think this is totally fine nowadays. We shouldn't need ForceRestore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-Infrastructure community-contribution Indicates that the PR has been added by a community member

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Restoring all packages step during the src/tests build

2 participants