Skip to content

Default compiler bootstrap config causes a warning to be shown #140928

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.

Sign up for GitHub

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

Jump to bottom
Closed
yotamofek opened this issue May 11, 2025 · 1 comment · Fixed by #140934
Closed

Default compiler bootstrap config causes a warning to be shown #140928

yotamofek opened this issue May 11, 2025 · 1 comment · Fixed by #140934
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Comments

@yotamofek
Copy link
Contributor

The enable-warnings = true line in src/bootstrap/defaults/bootstrap.compiler.toml causes the following warning to be shown every time x.py is invoked:

WARNING: `llvm.enable-warnings` has no effect with `llvm.download-ci-llvm`. Current value: Some(true), Expected value(s): None

Not sure what the correct fix here is, so opening up this issue instead of filing a PR to fix it :)
@yotamofek yotamofek added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) C-bug Category: This is a bug. labels May 11, 2025
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label May 11, 2025
@onur-ozkan
Copy link
Member

The enable-warnings = true line should be removed.

@onur-ozkan onur-ozkan added E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. C-bug Category: This is a bug. labels May 12, 2025
@yotamofek yotamofek mentioned this issue May 12, 2025
Merged
@bors bors closed this as completed in bc7512e May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Silence warning in default compiler bootstrap settings yotamofek/rust
3 participants
@yotamofek @onur-ozkan @rustbot