-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
tools no longer find librustc_driver #137469
Comments
I will check it today |
I tried this: and they seem to be working: $ ~/devspace/.other/rustc-builds $ ls ./build/host/stage1/bin
cargo-clippy cargo-fmt clippy-driver rustc rustfmt
$ ~/devspace/.other/rustc-builds $ ./build/host/stage1/bin/cargo-fmt --version
rustfmt 1.8.0-dev (bb2cc59a21 2025-02-23)
$ ~/devspace/.other/rustc-builds $ ./build/host/stage1/bin/rustfmt --version
rustfmt 1.8.0-dev (bb2cc59a21 2025-02-23)
$ ~/devspace/.other/rustc-builds $ ./build/host/stage1/bin/cargo-clippy --version
clippy 0.1.86 (bb2cc59a21 2025-02-23) How did you perform your test? |
with https://github.com/kennytm/rustup-toolchain-install-master I do a
and then |
It has the rpath and NEEDED, so the tool binary itself looks fine.
But the librustc_driver has a different hash..... somehow the tools are built against the wrong rustc. |
This feels related to #123384, maybe that tool staging bug messed up your new build logic? |
$ ~/devspace/.other/rustc-builds $ readelf -d build/host/stage2/bin/rustdoc
Dynamic section at offset 0xc051b0 contains 32 entries:
Tag Type Name/Value
0x0000000000000001 (NEEDED) Shared library: [librustc_driver-e09e126965159615.so]
0x0000000000000001 (NEEDED) Shared library: [libm.so.6]
0x0000000000000001 (NEEDED) Shared library: [libgcc_s.so.1]
0x0000000000000001 (NEEDED) Shared library: [libc.so.6]
0x0000000000000001 (NEEDED) Shared library: [ld-linux-x86-64.so.2]
0x000000000000001d (RUNPATH) Library runpath: [$ORIGIN/../lib]
$ ~/devspace/.other/rustc-builds $ ls build/host/stage2/lib
librustc_driver-e09e126965159615.so rustlib Regular builds seems fine, will look into |
Yeah, |
I wonder if we can have a little smoke test on CI
or something like that |
ohhhhh, your #137215 did #123384, that's very cool! nice. @matthiaskrgr yeah, having dist-smoke-tests would be useful. see #136822 when recently i686-pc-windows-gnu suddenly had x86-64 libraries :D |
I've prepared a revert in #137475 in case we don't find a fix for the bug in time, so there's no rush :). |
I found the bug and did the fix, just preparing the PR now :) |
avoid `compiler_for` for dist steps and force the current compiler Using `compiler_for` in dist steps was causing to install stage1 tools into the dist tarballs, which doesn't match with the stage2 compiler. Fixes rust-lang#137469
avoid `compiler_for` for dist tools and force the current compiler Using `compiler_for` in dist steps was causing to install stage1 tools into the dist tarballs, which doesn't match with the stage2 compiler. Fixes rust-lang#137469
avoid `compiler_for` for dist tools and force the current compiler Using `compiler_for` in dist steps was causing to install stage1 tools into the dist tarballs, which doesn't match with the stage2 compiler. Fixes rust-lang#137469
avoid `compiler_for` for dist tools and force the current compiler Using `compiler_for` in dist steps was causing to install stage1 tools into the dist tarballs, which doesn't match with the stage2 compiler. Fixes rust-lang/rust#137469
avoid `compiler_for` for dist tools and force the current compiler Using `compiler_for` in dist steps was causing to install stage1 tools into the dist tarballs, which doesn't match with the stage2 compiler. Fixes rust-lang/rust#137469
~/.cargo/bin/cargo +master fmt
~/.cargo/bin/cargo +master clippy
cc @onur-ozkan #137215
The text was updated successfully, but these errors were encountered: