-
Notifications
You must be signed in to change notification settings - Fork 1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
This adds MSVC builds built on Linux using wine. This requires some settings of tools and flags because the autotools support for MSVC is naturally somewhat limited. The advantage of this approach is that it is compatible with the our existing CI scripts, so there's no need to write a Windows CI script (in PowerShell or similar). A limitation of this approach is that it still relies on autotools and some less important parts of the MinGW-w64 toolchain, e.g., ar and nm. As such, the approach is most likely good enough to spot real miscompilations but it does not replicate native MSVC builds entirely. If we want to test building and running on Windows native (e.g., as supported by Cirrus CI) we could still do this in the future. Another feature of this approach is that contributors can simply use the docker image if they need a MSVC installation in a non-Windows environment. This commit also raises the TEST_ITERS for wine tasks to the default. The overhead of wine is negligible, so we can certainly afford the same number of iterations as for native Linux tests. This commit also improves the Dockerfile by grouping RUN commands according to Docker docs: https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#run
- Loading branch information
1 parent
36f9f04
commit 3a0ecee
Showing
3 changed files
with
105 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters