Add MingW support#550
Conversation
|
Thanks for the PR! I see there is no way to test the mingw config in the CI, could you update it? Tips: it might be faster to test the CI on your fork first than here (since GitHub continuously ask me to approve workflow runs somehow?) |
|
Hi there! Would you still like to add MinGW support to ArkScript? |
|
Hey @SuperFola, sorry for the inexcusably late reply. Back when I tried integrating MingW into the CI pipeline, I wasn't able to get it to work for the entire project, only for the interpreter and the REPL. I remember stumbling on the http module in particular, as that didn't compile due to it requiring OpenSSL, which itself was not available as a MingW package on any Ubuntu system. So unfortunately I think that adding and maintaining first-class support for MingW is unrealistic. Regarding the CI pipeline, it would only be possible to get it to work reliably by compiling without modules, but at that point the pain is not worth the gain, at least in my opinion; compiling in a VM or on a dedicated machine is less trouble. |
|
Hi again, thanks for coming back! Indeed, that would be quite complicated to maintain, plus it would increase CI compile times a lot if we have to compile openssl, or drop the modules compilation + modules' tests, which I would be against as supporting a compiler (imho) means supporting it entirely, not just one or two CMake targets. Is that okay if I close the PR? |
|
Sure, please go ahead and close the PR |
Description
Ark can currently not be cross-compiled from Linux to Windows using MingW. There are some minor incompatibilities across the code-base and missing relevant compilation options.
This PR enables compatibility with
x86_64-w64-mingw32-c++as the compiler, the changes consist of some required compilation switches, and some minor code changes.Checklist