-
Notifications
You must be signed in to change notification settings - Fork 555
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
[LLVM_jll] v13.0.1+3 on julia 1.8.5 throws an ArgumentError
#8856
Comments
I think this was fixed in 1.9, but for 1.8 JuliaPackaging/JLLWrappers.jl#35 (comment) |
I can try to make a PR to the LLVM_jll repo, is it just adding TOML as a dependency and rebuilding so we get v13.0.1+4? Or do I need to add it to MCAnalyzer.jl..? |
Is there anything that needs to be done for this in Yggdrasil, or should this be moved to a different repo? |
I have no idea - since the original issue is due to a JLL, I opened it here. I'm not familiar enough with the internals of a JLL package to say where/how this should be fixed. I'm just getting annoyed by the CI I set up for MCAnalyzer.jl that now tells me every week that this is still broken... |
As observed here: https://github.com/JuliaPerf/MCAnalyzer.jl/actions/runs/9406622087/job/25910502181#step:6:45
This seems to be because https://github.com/JuliaBinaryWrappers/LLVM_jll.jl/blob/LLVM-v13.0.1%2B3/.pkg/select_artifacts.jl uses TOML, but nothing puts that on the LOAD_PATH in 1.8.5 🤔
The text was updated successfully, but these errors were encountered: