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

azure: use matrix strategy to avoid configuration redundancy #5468

Closed
wants to merge 1 commit into from

Conversation

mback2k
Copy link
Member

@mback2k mback2k commented May 27, 2020

This also includes the following changes:

  • Use the same timeout for all jobs on Linux (60 minutes) and Windows (90 minutes)
  • Use CLI stable apt-get install -y instead of apt install which warns about that and run apt-get update first
  • Enable MQTT for Windows msys2 builds instead of legacy msys1 builds
  • Add ./configure --prefix parameter to the msys2 builds
  • The MSYSTEM environment variable is now preset inside the container images for the msys2 builds

@mback2k mback2k added the tests label May 27, 2020
@mback2k mback2k self-assigned this May 27, 2020
@bagder
Copy link
Member

bagder commented May 28, 2020

Sounds fine. We just need to make sure to update the CI-*pl scripts in the https://github.com/curl/stats repo once this lands to make them count the jobs correctly.

This also includes the following changes:

- Use the same timeout for all jobs on Linux (60 minutes)
  and Windows (90 minutes)
- Use CLI stable apt-get install -y instead of apt install
  which warns about that and run apt-get update first
- Enable MQTT for Windows msys2 builds instead of
  legacy msys1 builds
- Add ./configure --prefix parameter to the msys2 builds
- The MSYSTEM environment variable is now preset inside
  the container images for the msys2 builds
mback2k added a commit to curl/stats that referenced this pull request May 30, 2020
@mback2k
Copy link
Member Author

mback2k commented May 30, 2020

We just need to make sure to update the CI-*pl scripts in the https://github.com/curl/stats repo once this lands to make them count the jobs correctly.

I took the pleasure to update and prepare the stats scripts in advance. Hope that suits you well!

@mback2k mback2k closed this in 4bd0987 May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants