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

commented Oct 5, 2018

  • 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

This comment has been minimized.

Copy link
Member

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. =(

MarcelRaad added 6 commits Oct 7, 2018
AppVeyor: add remaining Visual Studio versions
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.
AppVeyor: add WinSSL builds
Use the oldest and latest Windows SDKs for them.
Also, remove all but one OpenSSL build.
AppVeyor: use Debug builds to run tests
This enables more tests.

@MarcelRaad MarcelRaad force-pushed the MarcelRaad:more_appveyor_builds branch from cca5833 to f0ae5d7 Oct 7, 2018

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
AppVeyor: add remaining Visual Studio versions
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
AppVeyor: add WinSSL builds
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
AppVeyor: use Debug builds to run tests
This enables more tests.

Closes curl#3104

@MarcelRaad MarcelRaad closed this in ae762e1 Oct 7, 2018

@MarcelRaad MarcelRaad deleted the MarcelRaad:more_appveyor_builds branch Oct 7, 2018

@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.
Projects
None yet
2 participants
You can’t perform that action at this time.