Skip to content

Change "error finalizing incremental compilation" text and emit it as a note, not a warning#154110

Merged
rust-bors[bot] merged 4 commits intorust-lang:mainfrom
lambdageek:fix/incr-compile-note
Mar 28, 2026
Merged

Change "error finalizing incremental compilation" text and emit it as a note, not a warning#154110
rust-bors[bot] merged 4 commits intorust-lang:mainfrom
lambdageek:fix/incr-compile-note

Conversation

@lambdageek
Copy link
Copy Markdown
Contributor

As mentioned in #151181 (comment) and #151181 (comment) the current message could be improved:

  1. Right now it displays as "warning: error ..." which is confusing (is it an error or a warning)
  2. It doesn't give the user a clear indication of what the consequences are
  3. The current build is successful. The next build might be slower

The new message is now

note: did not finalize incremental compilation session directory ...
  |
  = help: the next build will not be able to reuse work from this compilation

I started a zulip thread #t-compiler/incremental > Ergonomics of "error finalizing incremental session"

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

Labels

A-run-make Area: port run-make Makefiles to rmake.rs 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants