Skip to content

./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

Open
bjorn3 opened this issue Jun 7, 2023 · 5 comments
Open

./x.py check broken with vendored dependencies #112393

bjorn3 opened this issue Jun 7, 2023 · 5 comments
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

@bjorn3
Copy link
Member

bjorn3 commented Jun 7, 2023

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.

Checking stage0 gcc (aarch64-unknown-linux-gnu)
    Updating git repository `https://github.com/antoyo/gccjit.rs`
error: failed to get `gccjit` as a dependency of package `rustc_codegen_gcc v0.1.0 (/home/gh-bjorn3/rust/compiler/rustc_codegen_gcc)`

Caused by:
  failed to load source for dependency `gccjit`

Caused by:
  Unable to update https://github.com/antoyo/gccjit.rs#fe242b7e

Caused by:
  failed to fetch into: /home/gh-bjorn3/.cargo/git/db/gccjit.rs-13c2e290f2fb9e4d

Caused by:
  attempting to update a git repository, but --frozen was specified
Build completed unsuccessfully in 0:00:38

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.

@bjorn3 bjorn3 added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) C-bug Category: This is a bug. A-gcc Things relevant to the [future] GCC backend labels Jun 7, 2023
@bjorn3
Copy link
Member Author

bjorn3 commented Jun 7, 2023

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
@Kobzol
Copy link
Contributor

Kobzol commented Sep 27, 2023

@bjorn3 do you consider this to be solved after #114415?

@bjorn3
Copy link
Member Author

bjorn3 commented Sep 27, 2023

Yes, though vendoring will need to be implemented for it at one point.

@Kobzol
Copy link
Contributor

Kobzol commented Sep 27, 2023

Ok, then probably a separate issue is to implement vendoring for the gcc backend.

@Kobzol Kobzol closed this as completed Sep 27, 2023
@onur-ozkan
Copy link
Member

Re-opening the issue as it's linked with a FIXME note in bootstrap.

// FIXME: remove once https://github.com/rust-lang/rust/issues/112393 is resolved
if builder.build.config.vendor && self.backend == "gcc" {
println!("Skipping checking of `rustc_codegen_gcc` with vendoring enabled.");
return;

@onur-ozkan onur-ozkan reopened this May 28, 2024
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)
Projects
None yet
Development

No branches or pull requests

3 participants