Skip to content
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

Add Windows to CI matrix #350

Open
kazuho opened this issue Mar 26, 2021 · 3 comments
Open

Add Windows to CI matrix #350

kazuho opened this issue Mar 26, 2021 · 3 comments

Comments

@kazuho
Copy link
Member

kazuho commented Mar 26, 2021

By adding Windows to the flavor, we can prevent the build from becoming corrupted repeatedly on Windows.

See also: #349

@huitema
Copy link
Collaborator

huitema commented Mar 26, 2021

I am not sure that Travis has an option for running on Windows. In picoquic, I do that by running tests for Windows using appveyor. Would appveyor work for you, or do you prefer to use github actions?

@kazuho
Copy link
Member Author

kazuho commented Mar 26, 2021

I do not have a strong opinion, though I think we can have more eyes to maintain the CI integration if we use GitHub actions, as we use it for h2o already.

@huitema
Copy link
Collaborator

huitema commented Mar 28, 2021

I have started writing a github action that compiles the Visual Studio solution -- see PR #352. Currently this fails, because the build requires access to OpenSSL libraries and headers -- and the exe requires access to openssl dll.

I suppose I could write a script that downloads the whole tarball, as well as the required version of perl, compiles it, and puts it in a location that the windows build can use. But that's quite a bit of scripting... I wish someone has done that already, but i can't find it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants