-
Notifications
You must be signed in to change notification settings - Fork 13k
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
x86_64-pc-windows-gnu: ld: cannot find -lcredui and -lsecur32 after upgrading to rustc 1.26.0-nightly (521d91c6b 2018-03-14) #49044
Comments
@messense can you narrow down the date range when it broke any further (e.g., try 2018-03-08)? That would help a lot in figuring out what changed. (Or is that the next nightly after 2018-03-08...?) |
triage: P-high |
@messense I see, shame. I'm going to assign this to myself, but I'll have to figure out some way to reproduce first. If you're willing, it may be possible to use the per-PR bors builds to bisect this down, but I'm not sure if those are available for x86_64-pc-windows-gnu (cc @rust-lang/release) |
@messense This tool can install our intermediate builds, which are available for every PR that lands: https://crates.io/crates/rustup-toolchain-install-master @kennytm thinks it works on windows, but has never tried it. Think you could try to use it? To start, you might try 2789b06, which is the commit that 2018-03-07 builds and hence which ought to work. You would do something like:
|
@nikomatsakis I don't have any Windows machine/VM so I'll have to use AppVeyor to try it. https://ci.appveyor.com/project/laumann/compiletest-rs/build/1.0.39 |
So, this is amusing. The dependency on these libraries is pulled in -- and, I think, provided -- by cc @alexcrichton was there a winapi upgrade past few days? |
No such dependency was present in the 03-07 version. |
Oh no, rustup-toolchain-install-master does not build.
|
For Rust builds The fix is simple, make sure you always keep the MinGW import libraries bundled with Rust in sync with what |
@kennytm It seems that rustup-toolchain-install-master does not download cargo, thus
|
Thanks!
Is there a list somewhere of what we bundle? How is that controlled? (Do you know?)
Is there some reason we can't? |
Where the list of bundled libraries is currently set: Lines 201 to 229 in 6c70cd1
#47359 keeps track of which bundled import libraries we currently need.
It's entirely possible to write such a test. It's just that nobody has done so due to being a bit tricky. |
It ought to be possible to run run-pass-fulldep test suite in an environment that hasn’t the full mingw. This would prevent the tests from picking up dependencies from the CI’s mingw. |
@messense Thanks, strange that it requires a Although for this, it is better to use https://github.com/Mark-Simulacrum/bisect-rust to automatically find the regression commit. |
I've opened a PR at #49055 to extend the list we're shipping. |
If you have MSYS installed, then there is an ugly workaround: copy libcredui.a and libsecur32.a from /mingw64/x86_64-w64-mingw32/lib to .multirust\toolchains\nightly-x86_64-pc-windows-gnu\lib\rustlib\x86_64-pc-windows-gnu\lib . It worked for me when I tried to install clippy, but then I got hit by rust-lang/rust-clippy#2532 :( |
…matsakis rustbuild: Add more MinGW libraries to ship Closes rust-lang#49044
When is it going to land in nightly? Right now -- rustc 1.26.0-nightly (55c984e 2018-03-16) -- I have three(!) library missing when building clippy 0.0.188:
|
See #49080 for more info on |
@NovemberZulu nightlies are released at 00:00 UTC, so should be in 2-3 hours. |
UPDATE: Never mind, I got totally confused without |
rustbuild: Ship libsynchronization Hot on the heels of rust-lang#49044 comes similar issue with libsynchronization. Discovered while building clippy: ``` <skipped> Compiling serde_derive v1.0.33 error: linking with `gcc` failed: exit code: 1 <skipped> = note: ld: cannot find -lsynchronization ``` r? @nikomatsakis
https://ci.appveyor.com/project/laumann/compiletest-rs/build/1.0.38/job/o8rkn779qh7ios49
It builds fine with nightly-2018-03-07
https://ci.appveyor.com/project/laumann/compiletest-rs/build/1.0.38/job/rcfcfv3mfmx7h6t3
The relevant code here: Manishearth/compiletest-rs#108
The text was updated successfully, but these errors were encountered: