-
Notifications
You must be signed in to change notification settings - Fork 77
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
Build failure in msys2 in windows after updating from 1.1.9 to 1.1.10 #143
Comments
I think there is two PRs that could cause this: Of course, that doesn't explain what clippy would have to do with this, but it should be a good starting point. Maybe you could revert these PRs and try it locally to see if one of them is indeed causing the issue? In the meantime, I will see if running |
It looks like it doesn't reproduce with any project, This probably leaves us with you trying to apply a fix locally like suggested here or with providing more information on how to reproduce the issue. Besides, my top-tip is to undo #39 first. It's a single line, with great potential. |
This reverts commit 491bd1e. Fixes rust-lang#143.
Which |
Oh wow thank you for investigating it! I intended to try and bisect it later today when I was back in front of a pc 😅 |
I think it's important to fix the issue now as it was recently introduced. This buys time to see what it takes to use |
A new release was created that contains the fix. I truly hope we are back to a stable state now. |
Just a small detail about the The check for One could target |
Confirmed the new version works for me. Thanks again! |
Having a look at the build log in the original report, it looks like it's actually trying to build a vendored As a result, it's extremely strange that 39a51ea (a change in
There's a PR to change it, but the maintainers currently don't want to change it, as they're worried about breakages. When the ...and configures with On Windows when built with an MSVC host (because Otherwise, it can only use the system include directories, and relies on As a result, On other platforms that mostly isn't a problem, because there's normally an appropriate As for why the rollback "fixed it", by suspicion is that when I think it was the right call to do a roll-back, but while #39 triggered this issue, it isn't the root cause. I suspect that alexcrichton/curl-rust#486 would have fixed this where there's a system-installed |
I found where Which is built by Lines 93 to 94 in 0bb7bd4
Lines 135 to 137 in 0bb7bd4
So yes, |
Thanks so much @micolous for this awesome detective work! It's clearly beyond me, too, and I hope you will be able to orchestrate the involved crates into a better state overall. |
I thought (hoped) #141 would fix it but seemingly not.
When running
cargo clippy
on a project that indirectly depends on libz-sys, I can no longer build after libz-sys got updated to 1.1.10 (including 1.1.11). I get the following error:Downgrading to 1.1.9 (
cargo update -p libz-sys --precise 1.1.9
) makes it work as expected. Upgrading it again makes it break.Only running clippy seems to be affected, I can still do an actual build fine for some reason. I can also clippy fine in regular cmd.exe, so this is probably specific to using msys2. It worked before though! :(
The text was updated successfully, but these errors were encountered: