-
Notifications
You must be signed in to change notification settings - Fork 492
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
Update CUDA Linux GPG repository key [and compiler version]. #677
Conversation
ba1d466
to
d4ded9d
Compare
Additional commit replaces windows-2016 with windows-2019, because former was removed from Github Actions. |
The mingw error after merge is new. Should I look into it? I mean maybe how to fix it is already common knowledge... |
It appears to be Windows 2022-specific. Because if I force |
It looks like a miscompile. Maybe a rustc issue? If I try to run the
That NTSTATUS code is I haven't investigated further. |
Just in case, it's not like it fails to run any mingw32 executable. The prior tests do pass and then it fails |
Though you're showing |
I am replicating the failed CI test:
I don't know where it "ought" to be. I can only say where it is and that the binary is broken as it won't even load. |
Got it! Thanks! It ought to be one of the "unusual" tests in all.rs, "OptLink" or "asm". I'll try to triage... |
Hmmm.... I've masked the "OptLink", "asm" and "cpp" tests, and the test in question keeps failing. As mentioned it's mingw gcc, and by extension GNU ld, that does the linking. There is no reason to believe that most basic .c modules would be inconsistent with GNU ld. At the same time one can't say that it's between rustc and GNU ld, because linking pure Rust programs works. Or in other words, GNU ld does not get "confused" by rustc alone. Nor should it get confused by C alone. It gets "confused" specifically by a mixture of libraries compiled by rustc and cc... |
OK, the |
Just in case, all tests work with |
Ah, yeah, I do think mingw uses DWARF for exception handling. Only msvc targets use SEH currently. |
So the workaround for now would be to set the job to |
Well, whatever information is there to assist the exception handling is unlikely to be |
This was tested and proven to work, yes. |
Sure but I mean if it's using DWARF for exception handling then I would be very surprised if it wasn't also using DWARF for debugging symbols. |
Actions' windows-latest is equipped with newer mingw, which causes linking problems in mingw32 debug builds. See rust-lang#677 for details.
Opened #697 as temporary workaround. |
Actions' windows-latest is equipped with newer mingw, which causes linking problems in mingw32 debug builds. See rust-lang#677 for details.
BTW. Chris! Since you can reproduce it on your computer, could you check if setting CFLAGS and CXXFLAGS environment variables to |
Yep, those flags seem to fix it.
|
Yeah, it looks very much like mingw32 equivalent of #694, I'll leave a note there. Thanks! |
Nvidia rotated their repository signing key on April 27, 2022.