-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[CompilerSupportLibraries_jll] Upgrade to libraries from GCC 12 #45582
Conversation
I have no idea what's wrong with Windows (lots of undefined references to llvm functions at linking time for |
Alright, we have a problem. In https://github.com/JuliaPackaging/Yggdrasil/blob/7639eb0a7a1ec9d21cb8701925b4ec4f0257423e/C/CompilerSupportLibraries/build_tarballs.jl we always use the latest version of libgomp and libstdc++, independently from the version of libgfortran (this is for example important to solve the issue with undefined references to GLIBCXX symbols). However, GCC 12 changed the ABI of libgcc_s for x86_64-darwin, which is now called |
Do we know that they actually can't co-exist? 😈 |
Julia now builds on x86_64-darwin. Still linking errors on Windows for |
Not sure what the windows error is due to, but the macOS part looks good to me. |
Linking on windows now works (without having changed anything here), but Julia fails to bootstrap. What's wrong always with Windows? |
Right, this requires #45649, otherwise the Float16 tests are failing. Going back to Windows, the error now is:
which looks similar to issues mentioned in #45857 and #45641 (comment). If that's the case, it means Julia is picking the wrong libstdc++? I can't debug much myself anything related to Windows, so I'd need help from someone with access to a Windows box 🙂 |
What's the status of this? Still seems to have build problems. Is this release-blocking? |
I can't do much about Windows, I don't have a Windows box (and zero experience with Windows anyway). I'm surprised by the macOS failure, also because it was working before the last rebase, but haven't had the time to look into that. It's probably not a release blocker strictly speaking, but it'd be nice to get this in: just see at the people having problems with the old libstdc++ shipped by Julia in JuliaGL/GLFW.jl#198. |
Just doing a merge-up and rerun, given Mosé is on vacation for 3 weeks. Just in case it all goes green |
MacOS now built and I believe the test fail is a known FileWatching issue, but the windows issues remain and it's not obvious to me why |
missing Update: What is the full version number of our gcc12? Maybe we need this patch?
|
201f3a1
to
644a547
Compare
This commit message is a bit messy, and seems to claim to add debug checks and revert commits that aren't part of Julia. Please be conscientious when merging so that they are easily readable to others. |
I think this broke compilation with gcc <= 10 on Windows, due to something about the
If we then do
to get the lib files from the previous version back and then re-link julia (e.g. Perhaps we now need to avoid installing these |
We have been thinking about eliminating |
Usual memo to self:
stdlib/CompilerSupportLibraries_jll/Project.toml
CSL_NEXT_GLIBCXX_VERSION
indeps/csl.mk
make -f contrib/refresh_checksums.mk -j csl
libgcc_s
for x86_64-darwin changed, update all references to the version number wherever it's mentioned (I hope I got them all)Note 1: this lets us effectively test in CI #44829 is fixed
Note 2: I marked this to be backported to julia v1.6, but this will first require backporting #45249, which seems to be stalling.