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

Disable incremental on release builds #19218

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Feb 24, 2025

cc rust-lang/rust#137351

I do not know what intention behind this was, the introducing PR doesn't state why

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 24, 2025
@Veykril Veykril enabled auto-merge February 24, 2025 12:53
@lnicola
Copy link
Member

lnicola commented Feb 24, 2025

I always assumed it was so that it would compile faster. We disable it back for releases.

@Veykril
Copy link
Member Author

Veykril commented Feb 24, 2025

it would explain why the install xtask actions tends to give me gibberish linker errors every now and then though

@Veykril Veykril added this pull request to the merge queue Feb 24, 2025
Merged via the queue into rust-lang:master with commit b7e7893 Feb 24, 2025
9 checks passed
@Veykril Veykril deleted the push-vqsronpuvrnx branch February 24, 2025 13:24
@lnicola
Copy link
Member

lnicola commented Mar 3, 2025

You can really feel the difference here 😅.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants