Skip to content

Conversation

ia0
Copy link
Contributor

@ia0 ia0 commented Oct 8, 2025

Users who build core for the sole purpose of enabling panic_immediate_abort might expect "panic_immediate_abort is now a real panic strategy" to mean that setting panic = "immediate-abort" in Cargo.toml or -Cpanic=immediate-abort in RUSTFLAGS to be sufficient for migration. But this is not the case, core still needs to be built for those changes to take effect.

See #146974 for additional context.

See #147286 and rust-lang/cargo#16042 for the revelant tracking issues.

Users who build `core` for the sole purpose of enabling `panic_immediate_abort` might expect
"`panic_immediate_abort` is now a real panic strategy" to mean that setting `panic =
"immediate-abort"` in `Cargo.toml` or `-Cpanic=immediate-abort` in `RUSTFLAGS` to be sufficient for
migration. But this is not the case, `core` still needs to be built for those changes to take
effect.

See rust-lang#146974 for additional context.
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 8, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 8, 2025

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
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

@saethlin
Copy link
Member

saethlin commented Oct 8, 2025

I'm not libs so I can't actually approve this, unfortunately.

I wrote the original error because I was thinking of having added to the PanicStrategy enum. But this error isn't for people dealing with toolchain internals, it's migration assistance and should serve users.

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 9, 2025

📌 Commit 594f9c6 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 Oct 9, 2025
chenyukang added a commit to chenyukang/rust that referenced this pull request Oct 9, 2025
…crum

Clarify how to remediate the panic_immediate_abort error

Users who build `core` for the sole purpose of enabling `panic_immediate_abort` might expect "`panic_immediate_abort` is now a real panic strategy" to mean that setting `panic = "immediate-abort"` in `Cargo.toml` or `-Cpanic=immediate-abort` in `RUSTFLAGS` to be sufficient for migration. But this is not the case, `core` still needs to be built for those changes to take effect.

See rust-lang#146974 for additional context.

See rust-lang#147286 and rust-lang/cargo#16042 for the revelant tracking issues.
bors added a commit that referenced this pull request Oct 9, 2025
Rollup of 6 pull requests

Successful merges:

 - #143900 ([rustdoc] Correctly handle `should_panic` doctest attribute and fix `--no-run` test flag on the 2024 edition)
 - #147420 (Add diagnostic items for `pub mod consts` of FP types)
 - #147467 (Fix double warnings on `#[no_mangle]`)
 - #147470 (Clarify how to remediate the panic_immediate_abort error)
 - #147480 (Do not invalidate CFG caches in CtfeLimit.)
 - #147481 (format: some small cleanup)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Oct 9, 2025
Rollup of 12 pull requests

Successful merges:

 - #146568 (Port the implemention of SIMD intrinsics from Miri to const-eval)
 - #147373 (give a better example why `std` modules named like primitives are needed)
 - #147419 (bootstrap: add `Builder::rustc_cmd` that includes the lib path)
 - #147420 (Add diagnostic items for `pub mod consts` of FP types)
 - #147457 (specialize slice::fill to use memset when possible)
 - #147467 (Fix double warnings on `#[no_mangle]`)
 - #147470 (Clarify how to remediate the panic_immediate_abort error)
 - #147480 (Do not invalidate CFG caches in CtfeLimit.)
 - #147481 (format: some small cleanup)
 - #147488 (refactor: Remove `LLVMRustInsertPrivateGlobal` and `define_private_global`)
 - #147489 (Prefer to use repeat_n over repeat().take())
 - #147506 (compiletest: Isolate public APIs and minimize public surface area)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 8e363d3 into rust-lang:master Oct 9, 2025
10 checks passed
@rustbot rustbot added this to the 1.92.0 milestone Oct 9, 2025
rust-timer added a commit that referenced this pull request Oct 9, 2025
Rollup merge of #147470 - ia0:immediate-abort, r=Mark-Simulacrum

Clarify how to remediate the panic_immediate_abort error

Users who build `core` for the sole purpose of enabling `panic_immediate_abort` might expect "`panic_immediate_abort` is now a real panic strategy" to mean that setting `panic = "immediate-abort"` in `Cargo.toml` or `-Cpanic=immediate-abort` in `RUSTFLAGS` to be sufficient for migration. But this is not the case, `core` still needs to be built for those changes to take effect.

See #146974 for additional context.

See #147286 and rust-lang/cargo#16042 for the revelant tracking issues.
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. 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.

5 participants