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

[BETA 1.64] Only override published resolver when the workspace is different #10970

Merged
merged 2 commits into from
Aug 11, 2022

Conversation

epage
Copy link
Contributor

@epage epage commented Aug 10, 2022

What does this PR try to resolve?

Ensures when publishing a package that uses an implicit resolver = "1" to maintain an MSRV before the resolver key was stabilized the implicitness is retained rather than being turned into an explicit setting.

fixes #10954 (assuming that the workspace and its packages are configured with a consistent resolver)

Note: this is a backport of #10961 to beta

@rust-highfive
Copy link

r? @ehuss

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.64.0. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 10, 2022
@ehuss ehuss changed the title Only override published resolver when the workspace is different [BETA 1.64] Only override published resolver when the workspace is different Aug 11, 2022
@ehuss
Copy link
Contributor

ehuss commented Aug 11, 2022

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Aug 11, 2022

📌 Commit ee4bf08 has been approved by ehuss

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 11, 2022
@bors
Copy link
Contributor

bors commented Aug 11, 2022

⌛ Testing commit ee4bf08 with merge ded0899...

@bors
Copy link
Contributor

bors commented Aug 11, 2022

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing ded0899 to rust-1.64.0...

@bors bors merged commit ded0899 into rust-lang:rust-1.64.0 Aug 11, 2022
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Aug 11, 2022
@epage epage deleted the backport branch August 12, 2022 15:19
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 14, 2022
…mulacrum

Rollup of beta backports

This rolls up bumping stage0 to released stable and:

* Iterate generics_def_id_map in reverse order to fix P-critical issue rust-lang#100340
*  [BETA 1.64] Only override published resolver when the workspace is different rust-lang/cargo#10970

r? `@Mark-Simulacrum`
@ehuss ehuss added this to the 1.64.0 milestone Aug 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants