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

Removing appeveyor and windows integration tests with GitHub Actions, many warning fixes #1689

Merged
merged 132 commits into from
Mar 9, 2023

Conversation

dcoeurjo
Copy link
Member

@dcoeurjo dcoeurjo commented Feb 22, 2023

PR Description

  • Removes appeveyor
  • Adds windows-latest GitHub action
  • Adds a new option to enable Conan package manager at cmake step (-DENABLE_CONAN, needs Conan 1.57, super useful on windows for instance)
  • when using conan, all mandatory deps (zlib, boost) are installed automatically. Few optional ones are also installed fro the windows CI bot (so far: Cairo,FFTW3)
  • Fixes existing issues with find_package for optional deps.
  • On GitHub action windows-latest, cache is enabled for faster build (caching folder .conan)
  • Fixes few build issues on windows

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug mode.
  • All continuous integration tests pass (Github Actions & appveyor)

@dcoeurjo dcoeurjo mentioned this pull request Mar 9, 2023
6 tasks
@dcoeurjo
Copy link
Member Author

dcoeurjo commented Mar 9, 2023

Unless someone would like to review this one, I think I will self-merge it. Plenty of bug fixes due to the CI Debug scripts

@dcoeurjo dcoeurjo merged commit 77bd363 into master Mar 9, 2023
@dcoeurjo dcoeurjo deleted the conan branch March 9, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant