Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CP] Warn about old-style cache #52526

Closed
sigurdm opened this issue May 26, 2023 · 4 comments
Closed

[CP] Warn about old-style cache #52526

sigurdm opened this issue May 26, 2023 · 4 comments
Assignees
Labels
cherry-pick-approved Label for approved cherrypick request cherry-pick-merged Cherry-pick has been merged to the stable or beta branch. cherry-pick-review Issue that need cherry pick triage to approve merge-to-stable

Comments

@sigurdm
Copy link
Contributor

sigurdm commented May 26, 2023

Commit(s) to merge

dart-lang/pub#3921

Target

stable

Prepared changelist for beta/stable

https://dart-review.googlesource.com/c/sdk/+/305981

Issue Description

In dart 3.0 Pub on Windows stopped reading caches in C:\Users<user>\AppData\Roaming\Pub\Cache (in favor of C:\Users<user>\AppData\Local\Pub\Cache).

These have not been created by pub since dart 2.8, but have been used if existing until and including 2.19.

This has lead to some amount of confusion for users with an old cache. Especially if they had a %PATH% pointing to the old location, then they would invoke old binstubs of globally activated packages.

What is the fix

We detect the presence of the old cache, and print a warning, and instructions for how to proceed.

Why cherry-pick

It could help guide users to find the new cache.

Risk

low

Issue link(s)

#52386

Extra Info

No response

@sigurdm sigurdm added the cherry-pick-review Issue that need cherry pick triage to approve label May 26, 2023
@athomas
Copy link
Member

athomas commented May 26, 2023

LGTM

@itsjustkevin itsjustkevin added merge-to-stable cherry-pick-approved Label for approved cherrypick request labels May 26, 2023
@mit-mit
Copy link
Member

mit-mit commented May 30, 2023

@sigurdm is the warning the only change in pub in the new version compared to the current pinned version?

@sigurdm
Copy link
Contributor Author

sigurdm commented May 30, 2023

yes - unless I made a mistake it should be a true cherry pick of the warning

@mit-mit
Copy link
Member

mit-mit commented May 30, 2023

In that case SGTM

@itsjustkevin itsjustkevin added the cherry-pick-merged Cherry-pick has been merged to the stable or beta branch. label May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-approved Label for approved cherrypick request cherry-pick-merged Cherry-pick has been merged to the stable or beta branch. cherry-pick-review Issue that need cherry pick triage to approve merge-to-stable
Projects
None yet
Development

No branches or pull requests

6 participants