-
Notifications
You must be signed in to change notification settings - Fork 598
[OCaml] Add (cross-)compiler shards #11146
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
base: master
Are you sure you want to change the base?
Conversation
compiler_target = try | ||
parse(Platform, ARGS[end]) | ||
catch | ||
error("This is not a typical build_tarballs.jl! Must provide exactly one platform as the last argument!") | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I opted to go the GCC route and use the compiler target as the "target" for the build
(even though the toolchain we use during the build is the host toolchain for the most part, we do need both)
2a8a536
to
f9e5b2f
Compare
I noticed during this process that I also need to declare a runtime dependency on I don't know whether that's necessary for these shards, since that dependency should be handled pretty much automatically (unless someone specifically excludes the |
f9e5b2f
to
cb8db63
Compare
I think I have most of the bones in-place here @giordano should I start generating + uploading these shards, or is there something else to do first? |
Uhm, no, I think building and publishing the shards should be good now. |
Hm, I'm surprised CI doesn't work here. Does it maybe not use the manifest from the PR, but instead the one from master? |
There's a merge conflict in the Manifest, unfortunately the buildkite plugin ignores conflicts and moves on, instead of erroring out (been there already...). |
I'm sure there's plenty that needs fixing here, but the main build is working at least. I also had to move to a pre-release of v5.4.0 since OCaml's cross-compiler support was only recently shored up in Dec 2024 (great timing for us!)
companion PR's: JuliaPackaging/BinaryBuilderBase.jl#425 and JuliaPackaging/BinaryBuilder.jl#1382