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

Add VFX 2024 to CI #2053

Merged
merged 5 commits into from
Sep 24, 2024
Merged

Conversation

remia
Copy link
Collaborator

@remia remia commented Sep 21, 2024

Adding VFX 2024 Docker images from @jfpanisset to the CI workflow.

Also proposing a small bump in CMake minimum version to fix a warning appearing when using FetchContent_Populate on latest CMake releases.

Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
@doug-walker
Copy link
Collaborator

I've updated the build matrix spreadsheet to reflect these changes (scroll down):
https://docs.google.com/spreadsheets/d/1Iw5uRw6voB54wFpkSSOGrcAXkciZfx9nlBSVHz6N4P8/edit?usp=sharing

@doug-walker
Copy link
Collaborator

Per the discussion at the TSC meeting today, we will go ahead and merge this one. The failing CI checks are not relevant.

@doug-walker doug-walker merged commit 145bce2 into AcademySoftwareFoundation:main Sep 24, 2024
55 of 61 checks passed
@remia
Copy link
Collaborator Author

remia commented Sep 24, 2024

@doug-walker I forgot to mention that the names of the Linux jobs got updated so we will need to update Github required check list probably.

@remia remia deleted the add-vfx2024-ci branch September 24, 2024 13:37
@doug-walker
Copy link
Collaborator

It should be working (?), I updated the job names after I merged this. Please let me know if it needs more adjustment.

@remia
Copy link
Collaborator Author

remia commented Sep 24, 2024

Thanks @doug-walker, my bad, I was looking at a PR where the CI workflow was not triggered since the merge so it was missing required checks where the names changed, as expected.

chrisnashdisguise pushed a commit to chrisnashdisguise/OpenColorIO that referenced this pull request Oct 30, 2024
* Add VFX 2024 ASWF iamges to CI

Signed-off-by: Rémi Achard <remiachard@gmail.com>

* Bump CMake to 3.14 minimum to fix warning in FetchContent

Signed-off-by: Rémi Achard <remiachard@gmail.com>

* Fix error message formating

Signed-off-by: Rémi Achard <remiachard@gmail.com>

* Reduce warnings

Signed-off-by: Rémi Achard <remiachard@gmail.com>

---------

Signed-off-by: Rémi Achard <remiachard@gmail.com>
Co-authored-by: Doug Walker <doug.walker@autodesk.com>
Signed-off-by: Chris Nash <chris.nash@disguise.one>
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.

3 participants