Skip to content

Revert "Export derive at core::derive and std::derive"#159877

Open
fee1-dead wants to merge 1 commit into
rust-lang:mainfrom
fee1-dead-contrib:rev
Open

Revert "Export derive at core::derive and std::derive"#159877
fee1-dead wants to merge 1 commit into
rust-lang:mainfrom
fee1-dead-contrib:rev

Conversation

@fee1-dead

@fee1-dead fee1-dead commented Jul 25, 2026

Copy link
Copy Markdown
Member

View all comments

This reverts commit fec0998.

Temporarily addresses #159856 until we get the stability attribute to actually work...

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 25, 2026
@rustbot

rustbot commented Jul 25, 2026

Copy link
Copy Markdown
Collaborator

r? @clarfonthey

rustbot has assigned @clarfonthey.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: libs
  • libs expanded to 12 candidates
  • Random selection from 6 candidates

@fee1-dead

Copy link
Copy Markdown
Member Author

r? jhpratt

@rustbot rustbot assigned jhpratt and unassigned clarfonthey Jul 25, 2026
@jhpratt

jhpratt commented Jul 25, 2026

Copy link
Copy Markdown
Member

@bors r+ rollup

@rust-bors

rust-bors Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

📌 Commit c659cbd has been approved by jhpratt

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 25, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Jul 25, 2026
Revert "Export `derive` at `core::derive` and `std::derive`"

This reverts commit fec0998.

Temporarily addresses rust-lang#159856 until we get the stability attribute to actually work...
jhpratt added a commit to jhpratt/rust that referenced this pull request Jul 25, 2026
Revert "Export `derive` at `core::derive` and `std::derive`"

This reverts commit fec0998.

Temporarily addresses rust-lang#159856 until we get the stability attribute to actually work...
rust-bors Bot pushed a commit that referenced this pull request Jul 25, 2026
Rollup of 20 pull requests

Successful merges:

 - #138618 (Support using const pointers in asm `const` operand)
 - #157962 (Lower paths to functions in const args as ConstKind::Error)
 - #158404 (trait_solver: normalize next-gen region constraints)
 - #158709 (rustdoc: warn on improperly interleaved HTML/MD)
 - #159720 (document #[global_allocator] constraints)
 - #159732 (optimization: don't look for diagnostic/canonical items without rustc_attrs enabled)
 - #159738 (implement `CovariantUnsafeCell`)
 - #159740 (reuse regular exported_non_generic_symbols logic in Miri)
 - #159780 (check `extern "custom"` function pointers)
 - #159786 (rustdoc-js: ignore editor temp files in test folder discovery)
 - #159819 (std::sync::poison: disable auto_cfg on PoisonError::new)
 - #155388 (stepping into where-clauses during normalization may be productive)
 - #155914 (when bailing on ambiguity, don't force other results to ambig)
 - #159439 (Fix(lib/fs/win): Fall back on Win32 delete for `Dir::remove_file`)
 - #159676 (Update wasm-component-ld to 0.5.27)
 - #159730 (allow accessing the contents of UnsafeCell without going through get)
 - #159809 (Avoid `#[target_features]`)
 - #159826 (Remove redundant `#[rustc_paren_sugar]` feature gate)
 - #159853 (Updated expect messages for `CString` struct and method documentation)
 - #159877 (Revert "Export `derive` at `core::derive` and `std::derive`")
@theemathas

Copy link
Copy Markdown
Contributor

Surely this needs a crater run?

@theemathas

Copy link
Copy Markdown
Contributor

@bors try

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jul 25, 2026
Revert "Export `derive` at `core::derive` and `std::derive`"
@rust-bors rust-bors Bot added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Jul 25, 2026
@rust-bors

rust-bors Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

💔 Test for 18e53da failed: CI

@theemathas

Copy link
Copy Markdown
Contributor

GitHub Actions has encountered an internal error when running your job.

wut

@bors try

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jul 25, 2026
Revert "Export `derive` at `core::derive` and `std::derive`"
@rust-bors

rust-bors Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

💔 Test for fce1fe0 failed: CI

@theemathas

Copy link
Copy Markdown
Contributor

Github actions is down: https://www.githubstatus.com/incidents/pz7g535gbs6p

rust-bors Bot pushed a commit that referenced this pull request Jul 25, 2026
Rollup of 20 pull requests

Successful merges:

 - #138618 (Support using const pointers in asm `const` operand)
 - #157962 (Lower paths to functions in const args as ConstKind::Error)
 - #158404 (trait_solver: normalize next-gen region constraints)
 - #158709 (rustdoc: warn on improperly interleaved HTML/MD)
 - #159720 (document #[global_allocator] constraints)
 - #159732 (optimization: don't look for diagnostic/canonical items without rustc_attrs enabled)
 - #159738 (implement `CovariantUnsafeCell`)
 - #159740 (reuse regular exported_non_generic_symbols logic in Miri)
 - #159780 (check `extern "custom"` function pointers)
 - #159786 (rustdoc-js: ignore editor temp files in test folder discovery)
 - #159819 (std::sync::poison: disable auto_cfg on PoisonError::new)
 - #155388 (stepping into where-clauses during normalization may be productive)
 - #155914 (when bailing on ambiguity, don't force other results to ambig)
 - #159439 (Fix(lib/fs/win): Fall back on Win32 delete for `Dir::remove_file`)
 - #159676 (Update wasm-component-ld to 0.5.27)
 - #159730 (allow accessing the contents of UnsafeCell without going through get)
 - #159809 (Avoid `#[target_features]`)
 - #159826 (Remove redundant `#[rustc_paren_sugar]` feature gate)
 - #159853 (Updated expect messages for `CString` struct and method documentation)
 - #159877 (Revert "Export `derive` at `core::derive` and `std::derive`")
@theemathas

Copy link
Copy Markdown
Contributor

@bors try

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jul 25, 2026
Revert "Export `derive` at `core::derive` and `std::derive`"
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jul 25, 2026
Revert "Export `derive` at `core::derive` and `std::derive`"

This reverts commit fec0998.

Temporarily addresses rust-lang#159856 until we get the stability attribute to actually work...
rust-bors Bot pushed a commit that referenced this pull request Jul 25, 2026
Rollup of 23 pull requests

Successful merges:

 - #159720 (document #[global_allocator] constraints)
 - #159732 (optimization: don't look for diagnostic/canonical items without rustc_attrs enabled)
 - #159738 (implement `CovariantUnsafeCell`)
 - #159740 (reuse regular exported_non_generic_symbols logic in Miri)
 - #159780 (check `extern "custom"` function pointers)
 - #159786 (rustdoc-js: ignore editor temp files in test folder discovery)
 - #159819 (std::sync::poison: disable auto_cfg on PoisonError::new)
 - #155388 (stepping into where-clauses during normalization may be productive)
 - #155914 (when bailing on ambiguity, don't force other results to ambig)
 - #159204 (Add support to caller_location to rustc_public)
 - #159411 ([rustdoc] Correctly handle output options with --show-coverage)
 - #159439 (Fix(lib/fs/win): Fall back on Win32 delete for `Dir::remove_file`)
 - #159676 (Update wasm-component-ld to 0.5.27)
 - #159730 (allow accessing the contents of UnsafeCell without going through get)
 - #159809 (Avoid `#[target_features]`)
 - #159810 (Add tuple never coercion collection regression test)
 - #159826 (Remove redundant `#[rustc_paren_sugar]` feature gate)
 - #159853 (Updated expect messages for `CString` struct and method documentation)
 - #159877 (Revert "Export `derive` at `core::derive` and `std::derive`")
 - #159878 (bootstrap: Remove obsolete option `build.compiletest-use-stage0-libtest`)
 - #159882 (Update expect messages in library/alloc/boxed.rs and library/alloc/string.rs to follow the style guide)
 - #159891 (Split multiline derives into std/rustc macros)
 - #159895 (rustc-dev-guide subtree update)
@theemathas

Copy link
Copy Markdown
Contributor

Until crater is done,

@bors r-

@rust-bors rust-bors Bot removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 25, 2026
@rust-bors

rust-bors Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

This pull request was unapproved.

This PR was contained in the following rollups:

View changes since this unapproval

@rust-bors

rust-bors Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 011c518 (011c518242557d21690f7936b5812318dc2fec47)
Base parent: da86f4d (da86f4d0726be475afbbffe40cb2f65741c51ad3)

@theemathas

Copy link
Copy Markdown
Contributor

@craterbot check p=2

@craterbot

Copy link
Copy Markdown
Collaborator

👌 Experiment pr-159877 created and queued.
🤖 Automatically detected try build 011c518
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 25, 2026
@fee1-dead

Copy link
Copy Markdown
Member Author

I'm not sure if this needs a crater run. In any case we still need to revert it because it is wrongly stabilized.

@craterbot

Copy link
Copy Markdown
Collaborator

🚧 Experiment pr-159877 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@nia-e nia-e added the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Jul 26, 2026
@nia-e

nia-e commented Jul 26, 2026

Copy link
Copy Markdown
Member

cc @rust-lang/libs-api - i'd be inclined to just r+ this regardless of breakage since the stabilisation was unintentional, but clearly there's disagreement on this point. do we need to fcp this?

@theemathas

Copy link
Copy Markdown
Contributor

@nia-e If you want to just merge this, I suppose that's fine with me.

@theemathas theemathas added the relnotes Marks issues that should be documented in the release notes of the next release. label Jul 26, 2026
@jhpratt

jhpratt commented Jul 26, 2026

Copy link
Copy Markdown
Member

imo the crater run should be advisory — let people know that they're depending on something unintentionally. The result shouldn't change the fact that this PR should be merged.

@nia-e

nia-e commented Jul 26, 2026

Copy link
Copy Markdown
Member

the alternative is we FCP-stabilise core::derive retroactively, which is also possible ^^

@jhpratt

jhpratt commented Jul 26, 2026

Copy link
Copy Markdown
Member

True; I'm not opposed to that. There is literally nothing that could change or be unresolved given that it's only a re-export.

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

Labels

I-libs-api-nominated Nominated for discussion during a libs-api team meeting. relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-crater Status: Waiting on a crater run to be completed. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants