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

Revert "Stabilize $$ in Rust 1.63.0" #99435

Merged
merged 1 commit into from
Jul 19, 2022

Conversation

CAD97
Copy link
Contributor

@CAD97 CAD97 commented Jul 18, 2022

This mechanically reverts commit 9edaa76, the one commit from #95860.

#99035; the behavior of $$crate is potentially unexpected and not ready to be stabilized. #99193 attempts to forbid $$crate without also destabilizing $$ more generally.

@rustbot modify labels +T-compiler +T-lang +P-medium +beta-nominated +relnotes

(applying the labels I think are accurate from the issue and alternative partial revert)

cc @Mark-Simulacrum

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jul 18, 2022
@rustbot
Copy link
Collaborator

rustbot commented Jul 18, 2022

Error: Label P-medium can only be set by Rust team members

Please file an issue on GitHub at triagebot if there's a problem with this bot, or reach out on #t-infra on Zulip.

@rust-highfive
Copy link
Collaborator

r? @cjgillot

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 18, 2022
@CAD97 CAD97 mentioned this pull request Jul 18, 2022
@CAD97
Copy link
Contributor Author

CAD97 commented Jul 18, 2022

Trying again...

@rustbot modify labels +T-compiler +T-lang +beta-nominated +relnotes

@rustbot
Copy link
Collaborator

rustbot commented Jul 18, 2022

Error: Label beta-nominated can only be set by Rust team members

Please file an issue on GitHub at triagebot if there's a problem with this bot, or reach out on #t-infra on Zulip.

@CAD97
Copy link
Contributor Author

CAD97 commented Jul 18, 2022

Trying again...

@rustbot modify labels +T-lang

@rustbot rustbot added the T-lang Relevant to the language team, which will review and decide on the PR/issue. label Jul 18, 2022
@workingjubilee workingjubilee added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jul 18, 2022
@Mark-Simulacrum
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jul 18, 2022

📌 Commit c0569f2 has been approved by Mark-Simulacrum

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 Jul 18, 2022
@workingjubilee workingjubilee added relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jul 18, 2022
@compiler-errors compiler-errors 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 Jul 18, 2022
@workingjubilee
Copy link
Member

workingjubilee commented Jul 18, 2022

Whoops, I have no idea how the S-waiting-on-* tags got changed by me. I guess fiddling with the tags in the GH UI effectively did a PUT request.

bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 19, 2022
…askrgr

Rollup of 9 pull requests

Successful merges:

 - rust-lang#98028 (Add E0790 as more specific variant of E0283)
 - rust-lang#99384 (use body's param-env when checking if type needs drop)
 - rust-lang#99401 (Avoid `Symbol` to `&str` conversions)
 - rust-lang#99419 (Stabilize `core::task::ready!`)
 - rust-lang#99435 (Revert "Stabilize $$ in Rust 1.63.0")
 - rust-lang#99438 (Improve suggestions for `NonZeroT` <- `T` coercion error)
 - rust-lang#99441 (Update mdbook)
 - rust-lang#99453 (:arrow_up: rust-analyzer)
 - rust-lang#99457 (use `par_for_each_in` in `par_body_owners` and `collect_crate_mono_items`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 19932a5 into rust-lang:master Jul 19, 2022
@rustbot rustbot added this to the 1.64.0 milestone Jul 19, 2022
@CAD97
Copy link
Contributor Author

CAD97 commented Jul 19, 2022

Will a team member do the beta backport or should I go ahead and cherry-pick this onto the beta branch (assuming it gets beta-approved)?

@ehuss
Copy link
Contributor

ehuss commented Jul 19, 2022

@CAD97 Can you also post a revert of rust-lang/reference#1192 to the reference repo?

As for beta process, someone on the team first needs to mark this with the beta-accepted Accepted for backporting to the compiler in the beta channel. label. Then someone (usually on the release team, but anyone can post it) rolls up multiple beta-accepted PRs roughly once a week or two. https://forge.rust-lang.org/release/backporting.html contains more details.

@pnkfelix
Copy link
Member

@rustbot label: +beta-accepted

@rustbot rustbot added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Jul 21, 2022
@ehuss ehuss removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jul 22, 2022
@ehuss ehuss modified the milestones: 1.64.0, 1.63.0 Jul 22, 2022
ehuss added a commit to ehuss/rust that referenced this pull request Jul 22, 2022
This is in preparation to merge rust-lang#99435
which will require the toolstate to be passing.

1 commits in 9fce337a55ee4a4629205f6094656195cecad231..dd07eda38b15f756d60556b7bd7eed6d8db3eeca
2022-06-22 13:59:28 -0700 to 2022-07-21 19:05:26 -0700
- This reverts commit 151b8196a30b1c00cb422e25f459dd4354043a6a, reversing (rust-lang/reference#1192)
ehuss pushed a commit to ehuss/rust that referenced this pull request Jul 22, 2022
…=Mark-Simulacrum

Revert "Stabilize $$ in Rust 1.63.0"

This mechanically reverts commit 9edaa76, the one commit from rust-lang#95860.

rust-lang#99035; the behavior of `$$crate` is potentially unexpected and not ready to be stabilized. rust-lang#99193 attempts to forbid `$$crate` without also destabilizing `$$` more generally.

`@rustbot` modify labels +T-compiler +T-lang +P-medium +beta-nominated +relnotes

(applying the labels I think are accurate from the issue and alternative partial revert)

cc `@Mark-Simulacrum`
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 22, 2022
[beta] Beta 1.63 backports

* Reference: Revert $$ macro_metavar rust-lang/reference#1192
* Revert "Stabilize $$ in Rust 1.63.0" rust-lang#99435
* rustdoc: avoid inlining items with duplicate `(type, name)` rust-lang#99344
* Do not call `check_expr` twice in `check_compatible` rust-lang#99397
@CAD97 CAD97 deleted the revert-dollar-dollar-crate branch July 25, 2022 04:53
@CAD97
Copy link
Contributor Author

CAD97 commented Jul 25, 2022

reference revert happened: rust-lang/reference#1242

cedricschwyter added a commit to KyrillGobber/huehuehue that referenced this pull request Jul 1, 2023
refactor:higher-order http method macro

we want to switch to rust nightly to be able to make use of rust
metavariable expansions as defined by RFC rust-lang/rfcs#3086 and
as tracked by rust-lang/rust#83527. other references include
rust-lang/rust#99035.

this feature was stabilized in 1.63, then unstabilized again in
rust-lang/rust#99435 and is now only available in rust nightly, awaiting
restabilization. however, the feature is stable enough for our use case,
which is why i'm going ahead and enabling it.
cedricschwyter added a commit to KyrillGobber/huehuehue that referenced this pull request Jul 1, 2023
refactor:higher-order http method macro

we want to switch to rust nightly to be able to make use of rust
metavariable expansions as defined by RFC rust-lang/rfcs#3086 and
as tracked by rust-lang/rust#83527. other references include
rust-lang/rust#99035.

this feature was stabilized in 1.63, then unstabilized again in
rust-lang/rust#99435 and is now only available in rust nightly, awaiting
restabilization. however, the feature is stable enough for our use case,
which is why i'm going ahead and enabling it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-lang Relevant to the language team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants