-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Compiler overrides #5001
Compiler overrides #5001
Conversation
decided on just adding options in the constructor, think it's ok and a lot less complicated than adding yet another properties thing |
Based on https://github.com/compiler-explorer/infra/actions/runs/4968591553/jobs/8891278478 The following argument parsers seem broken:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More WIP review notes (have to stop now as ran out of time for now; only thing I didn't look at is compiler-overrides.ts
stdin: string; | ||
}; | ||
|
||
export type CompilationRequestOptions = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for busting these out 👍🏻
override updateState() { | ||
const state = this.getCurrentState(); | ||
this.container.setState(state); | ||
this.compilerShared.updateState(state); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neat
now live! |
Thanks a ton for this @partouf -- very useful! |
Compiler Explorer added "Compiler Overrides" [1], which for Rust include setting by default the 2021 edition. Since `rustc` does not allow that, the old link has stopped working. Reported to Compiler Explorer at [2]. Link: compiler-explorer/compiler-explorer#5001 [1] Link: compiler-explorer/compiler-explorer#5429 [2] Reported-by: Dimitri Belopopsky <dimitri@belopopsky.com> Closes: https://rust-for-linux.zulipchat.com/#narrow/stream/288089-General/topic/CE.20link.20on.20rust-for-linux.20site/near/387667888 Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
PR Includes #4318 and #4999 and #5000
Adds:
Fixes #3765
Fixes #4151
Fixes #3154
Fixes #2724
Fixes #1966
Possible to address suggestions in #3100