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

AppVeyor: test more build combinations #3104

Closed
wants to merge 6 commits into from

Conversation

MarcelRaad
Copy link
Member

  • add Visual Studio 2008 and 2010 builds
  • add two WinSSL builds: with oldest compiler and Windows SDK (VS 2008) and latest compiler and Windows SDK (VS 2015)
  • add HTTP_ONLY build
  • run tests with ENABLE_DEBUG
  • reduce number of OpenSSL builds to one

No tests are run for the WinSSL and OpenSSL builds yet because of curl-config test failures similar to #1262. For all other builds, the tests are run.

No VS 2017 builds yet as 1) it requires a non-default image with a limited software environment I have to test yet and 2) it is still a moving target regarding compiler warnings and standards conformance (this probably affects C++ much more than C, though).

@bagder
Copy link
Member

bagder commented Oct 6, 2018

Test 2100 proves itself to be flaky on travis, which is very sad and feels wrong since its a rather new test and the only command line test of DoH we have. =(

This adds Visual Studio 9 and 10 builds.
There's no 64-bit VC9 compiler on AppVeyor, so use it as the Win32
build. Also, VC9 cannot be used for running the test suite.
Use the oldest and latest Windows SDKs for them.
Also, remove all but one OpenSSL build.
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Oct 7, 2018
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Oct 7, 2018
This adds Visual Studio 9 and 10 builds.
There's no 64-bit VC9 compiler on AppVeyor, so use it as the Win32
build. Also, VC9 cannot be used for running the test suite.

Closes curl#3104
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Oct 7, 2018
Use the oldest and latest Windows SDKs for them.
Also, remove all but one OpenSSL build.

Closes curl#3104
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Oct 7, 2018
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Oct 7, 2018
This enables more tests.

Closes curl#3104
@MarcelRaad MarcelRaad closed this in ae762e1 Oct 7, 2018
@MarcelRaad MarcelRaad deleted the more_appveyor_builds branch October 7, 2018 13:31
@lock lock bot locked as resolved and limited conversation to collaborators Jan 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants