[WIP] try to support more wheels - #1149
Conversation
sobolevn
left a comment
There was a problem hiding this comment.
I think that x86_64 macos would be the only reasonable thing to support?
I am also not a fan of the CI speed. All jobs take 10+ minutes to test these platforms. Which is a long time :(
There was a problem hiding this comment.
Do we really need these platforms? There was 1 real user request for these platforms: #788
I am asking, because this is not free for us:
- We have to maintain all the build infra
- We have to run CI for this regularly
I think that msgspec can be compiled locally quite easily and quite efficently. So, proving wheels is not 100% required for it to work.
Moreover, there's not much platform specific code in msgspec for us to test it heavily.
There was a problem hiding this comment.
Yes, this will increase the CI time, but for people who use these targets, installing the wheel will be much faster than compiling it on the spot
There was a problem hiding this comment.
I guess we can compile manylinux and musslinux on different GitHub Runners
There was a problem hiding this comment.
Agree with @sobolevn.
I'd also be cautious to add more platforms on a hunch. If there's a real demand for it, by which I mean at least a ticket asking for a specific platform with a good reason, I'm all for it. But otherwise, maybe let's hold off with it a bit.
There was a problem hiding this comment.
In theory, without cibuildwheel, we could skip using QEMU, use zig cc instead, and compile the wheels on an x86 platform, but we would still need to test them, which takes about ~100+ seconds per wheel.
If we remove the musllinux wheels for s390x, ppc64le, and armv7l and keep what's in this PR, the build time will remain roughly the same - less than ~10 minutes per GitHub Runner.
This reverts commit a0cc627.

The diff is large because of:
actions-up --yes --min-age 0I'll revert these changes later