-
-
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
Resurrect libunwind patches #45189
Resurrect libunwind patches #45189
Conversation
@giordano this allows the patches to apply to 1.8.0-julia3, but then immediately afterwards, a new error:
This is never-ending… |
But why are patches inside the source repo not the same as the patches inside the builder software (if I understand correctly what Yggdrasil is)? |
7d96257
to
58dac77
Compare
Two more patches, yet another build failure:
There is no |
The answer is above: the makefile passes an explicit path to |
Shouldn't there be a dependency so that llvm is built before llvmunwind? I don't find it, and I think it would make sense. In @giordano any advice? I'm really trying to help, but the build system is so complex and so fragile, it's hard to know. |
So, with:
I can actually compile Julia 1.8.0-rc3 from source, on aarch64-apple-darwin21, with its own LLVM/libunwind. It passes the previous sore point (
It's been stuck there for a long while. It was using CPU at the beginning of this process, reached 25, then crashed:
The crash report is the following:
|
You can change julia/contrib/generate_precompile.jl Line 245 in 862018b
stdout to see if the print out reveals anything.
|
|
Under lldb I get a slightly better backtrace:
We're building against system libgit2, so maybe that's another non upstreamed patch. |
I hoped this dependency would allow to install llvm before llvmunwind is configured:
but it does not work reliably: it builds llvmunwind after llvm is built, but not necessarily installed (so |
I used the exact same build, but without the system libgit2 (removed
One step forward, two steps back. |
OK, I've now pushed a clean PR that fixes the missing patches issue, and allows compilation to proceed further. It does not fix problems that arise later, but it should be safe to commit and fix one issue at a time… @giordano could you review, or indicate who has review/commit privileges for the build system? |
Testing in progress…
I hope this can fix #44499