Skip to content

Commit 6353350

Browse files
authored
Rollup merge of #147830 - Noratrieb:reword-unstable-fingerprint-ice, r=jackh726
Reword unstable fingerprints ICE to ask for reproduction When the unstable fingerprints error was added, Rust was on fire, and we needed a quick way for people to sort of understand what's going on, follow the tracking issue, and leave some information without overwhelming the issue tracker and focusing on getting their code working. This is what motivated the previous message. It called this a "known issue", provided help on how to fix it, and only secondarily asked for a bug report. This is no longer true. These days incremental compilation is fairly solid and these issues are supposed to be rare, we expect *none* of them to exist (but obviously know that's not true). As such, it's time to reword this message. Recently someone mentioned how they didn't bother reporting this issue because it said that it was a "known issue", and I only got awareness of their problem because they complained about all the rustc-ice files hanging around their directories (#147825 (comment)). This is not at all what we want, we want reports from people, ideally with a reproduction. To get this, I reworded the error. It now explicitly asks for a reproduction (and explaining what that means) and no longer calls it a "known issue". It also does not link to the tracking issue anymore, because I don't think this tracking issue is useful. It should probably be closed. I still mention the workaround, but explicitly call it a "workaround". People should report a reproduction and only *then* use the workaround.
2 parents 98632ac + d66647b commit 6353350

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

compiler/rustc_query_system/messages.ftl

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,11 @@ query_system_cycle_stack_single = ...which immediately requires {$stack_bottom}
1616
query_system_cycle_usage = cycle used when {$usage}
1717
1818
query_system_increment_compilation = internal compiler error: encountered incremental compilation error with {$dep_node}
19-
.help = This is a known issue with the compiler. Run {$run_cmd} to allow your project to compile
2019
2120
query_system_increment_compilation_note1 = please follow the instructions below to create a bug report with the provided information
22-
query_system_increment_compilation_note2 = see <https://github.com/rust-lang/rust/issues/84970> for more information
21+
query_system_increment_compilation_note2 = for incremental compilation bugs, having a reproduction is vital
22+
query_system_increment_compilation_note3 = an ideal reproduction consists of the code before and some patch that then triggers the bug when applied and compiled again
23+
query_system_increment_compilation_note4 = as a workaround, you can run {$run_cmd} to allow your project to compile
2324
2425
query_system_overflow_note = query depth increased by {$depth} when {$desc}
2526

compiler/rustc_query_system/src/error.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,9 +69,10 @@ pub(crate) struct Reentrant;
6969

7070
#[derive(Diagnostic)]
7171
#[diag(query_system_increment_compilation)]
72-
#[help]
7372
#[note(query_system_increment_compilation_note1)]
7473
#[note(query_system_increment_compilation_note2)]
74+
#[note(query_system_increment_compilation_note3)]
75+
#[note(query_system_increment_compilation_note4)]
7576
pub(crate) struct IncrementCompilation {
7677
pub run_cmd: String,
7778
pub dep_node: String,

0 commit comments

Comments
 (0)