We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Commit 418b1fa broke builds on Windows as llvm-config has a .exe extension but
llvm-config
.exe
rust/src/bootstrap/native.rs
Lines 163 to 164 in 418b1fa
Patching line 163 with
let llvm_config = llvm_root.join("bin").join(exe("llvm-config", builder.config.build));
fixes the build.
The text was updated successfully, but these errors were encountered:
@BlaCoiso do you want to make a PR with that patch? :)
Sorry, something went wrong.
Rollup merge of rust-lang#98556 - BlaCoiso:patch-1, r=jyn514
31ec973
Fix builds on Windows (closes rust-lang#98546) closes rust-lang#98546
Auto merge of rust-lang#98566 - matthiaskrgr:rollup-43etyls, r=matthi…
8e52fa8
…askrgr Rollup of 5 pull requests Successful merges: - rust-lang#97389 (Improve memory ordering diagnostics) - rust-lang#97780 (Check ADT field is well-formed before checking it is sized) - rust-lang#98530 (compiletest: add issue number param to `known-bug`) - rust-lang#98556 (Fix builds on Windows (closes rust-lang#98546)) - rust-lang#98561 (Fix spelling in SAFETY comment) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
15abd82
Successfully merging a pull request may close this issue.
Commit 418b1fa broke builds on Windows as
llvm-config
has a.exe
extension butrust/src/bootstrap/native.rs
Lines 163 to 164 in 418b1fa
does not take that into account, resulting in a file not found error and bootstrap panicking.
Patching line 163 with
fixes the build.
The text was updated successfully, but these errors were encountered: