-
Notifications
You must be signed in to change notification settings - Fork 13.3k
./x.py check broken with vendored dependencies #112393
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
Labels
A-gcc
Things relevant to the [future] GCC backend
C-bug
Category: This is a bug.
T-bootstrap
Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Comments
cc @antoyo |
matthiaskrgr
added a commit
to matthiaskrgr/rust
that referenced
this issue
Aug 3, 2023
… r=ozkanonur Skip checking of `rustc_codegen_gcc` with vendoring enabled `rustc_codegen_gcc` currently cannot be vendored, which [breaks](rust-lang#112393) `x.py check` with vendoring enabled. Until the vendoring issue is resolved, it would be nice if `x.py check` could succeed (and just skip `gcc`) with `vendor = true`. With this PR, it does. Related issue: rust-lang#112393 r? bootstrap
github-actions bot
pushed a commit
to rust-lang/miri
that referenced
this issue
Aug 4, 2023
Skip checking of `rustc_codegen_gcc` with vendoring enabled `rustc_codegen_gcc` currently cannot be vendored, which [breaks](rust-lang/rust#112393) `x.py check` with vendoring enabled. Until the vendoring issue is resolved, it would be nice if `x.py check` could succeed (and just skip `gcc`) with `vendor = true`. With this PR, it does. Related issue: rust-lang/rust#112393 r? bootstrap
Yes, though vendoring will need to be implemented for it at one point. |
Ok, then probably a separate issue is to implement vendoring for the gcc backend. |
Re-opening the issue as it's linked with a FIXME note in bootstrap. rust/src/bootstrap/src/core/build_steps/check.rs Lines 275 to 278 in 7717a30
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-gcc
Things relevant to the [future] GCC backend
C-bug
Category: This is a bug.
T-bootstrap
Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
I tried to run
./x.py check
after vendoring all dependencies.I expected to see this happen: It passes.
Instead, this happened: It tries to check rustc_codegen_gcc, despite the dependencies of cg_gcc not being vendored.
Meta
https://github.com/antoyo/gccjit.rs is GPL-3.0 licensed just like libgccjit. I don't know if there would be any issues with adding it to the rustc-src tarball.
The text was updated successfully, but these errors were encountered: