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

Windows CI fixes #2796

Merged
merged 6 commits into from Dec 28, 2020
Merged

Windows CI fixes #2796

merged 6 commits into from Dec 28, 2020

Conversation

debaetsd
Copy link
Contributor

@debaetsd debaetsd commented Dec 28, 2020

Description

This PR fixes some issues found when trying to enable CI on Windows.
With these fixes, the Windows CI is running yet only about 70% of the tests pass.
Note that this PR does not enable CI nor fixes the failing tests.
I created issue #2797 to keep track of the failing tests.
Once all tests are stable, we can enable the CI for Windows.

Tests

Any existing CI is still running/passing.

Checklist:

  • I have read the contribution guidelines.
  • If this is more extensive than a small change to existing code, I
    have previously submitted a Contributor License Agreement
    (individual, and if there is any way my
    employers might think my programming belongs to them, then also
    corporate).
  • I have updated the documentation, if applicable.
  • I have ensured that the change is tested somewhere in the testsuite
    (adding new test cases if necessary).
  • My code follows the prevailing code style of this project.

@debaetsd debaetsd changed the title Winci Windows CI fixes Dec 28, 2020
Copy link
Collaborator

@lgritz lgritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great steps forward! LGTM.

@lgritz lgritz merged commit bb7b04c into AcademySoftwareFoundation:master Dec 28, 2020
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request Jan 4, 2021
* fixes to make vs2019 CI run. Not passing yet as some tests fail

* added explicit ctest timeout (180sec) + misc fixes to run CI local

* disable Windows VS2019 tests again as they are not passing yet

Co-authored-by: Dieter De Baets <dieter.baets@unity3d.com>
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

Successfully merging this pull request may close these issues.

None yet

2 participants