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

Update CI with vfx2023 Linux jobs. #1475

Merged
merged 1 commit into from
Jul 9, 2023

Conversation

xlietz
Copy link
Contributor

@xlietz xlietz commented Jun 30, 2023

Below are a list of the CI Linux builds in this PR and relate Imath PR (will add link in comment).

Questions for both Imath and OpenEXR :

  • Do we still need c++14 builds for 2023? Imath had them for 2022, OpenEXR did not.
  • Do we need static debug builds for 2023? OpenEXR had them for 2022, Imath did not.
  • Do we need clang builds for prior years? Imath had them for 2022, OpenEXR did not.
  • Any other builds I should drop or add?
  • Should we be using Clang 14.0 or 15.0 for 2023? AFAICT the docker image has both.

Imath
2023 gcc 11.2.1 python 3.10.9 C++17 - Shared Release
2023 gcc 11.2.1 python 3.10.9 C++17 - Shared Debug
2023 gcc 11.2.1 python 3.10.9 C++17 - Static Release
2023 gcc 11.2.1 python 3.10.9 C++14 - Static Release
2023 clang ? python 3.10.9 C++17 - Shared Release
2023 clang ? python 3.10.9 C++17 - Shared Debug
2023 clang ? python 3.10.9 C++17 - Static Release
2023 clang ? python 3.10.9 C++14 - Static Debug
2022 gcc 9.3.1 python 3.9.7 C++17 - Shared Release
2022 clang 10.4 python 3.9.7 C++17 - Shared Release
2021 gcc 9.3.1 python 3.7.9 C++17 - Shared Release
2021 clang 10.4 python 3.7.9 C++17 - Shared Release
2020 gcc 6.3.1 python 3.7.3 C++14 - Shared Release
2020 clang 7.8 python 3.7.3 C++14 - Shared Release
2019 gcc 6.3.1 python 3.7.3 C++11 - Shared Release

OpenEXR
2023 gcc 11.2.1 C++17 - Shared Release
2023 gcc 11.2.1 C++17 - Shared Release, Threads OFF
2023 gcc 11.2.1 C++17 - Shared Debug
2023 gcc 11.2.1 C++17 - Static Release
2023 gcc 11.2.1 C++17 - Static Debug
2023 clang ? C++17 - Shared Release
2023 clang ? C++17 - Shared Debug
2023 clang ? C++17 - Static Release
2023 clang 10.4 C++17 - Static Debug
2022 gcc 9.3.1 C++17 - Shared Release
2021 gcc 9.3.1 C++17 - Shared Release
2020 gcc 6.3.1 C++14 - Shared Release
2019 gcc 6.3.1 C++11 - Shared Release

@xlietz
Copy link
Contributor Author

xlietz commented Jun 30, 2023

Imath PR for 2023 CI updates:
AcademySoftwareFoundation/Imath#332

@meshula
Copy link
Contributor

meshula commented Jul 1, 2023

  • clang 14/15 are meaningful to xcode targets we support.
  • all compilers specified in 2023 vfxplatform support c++ 17, so dropping 14 should be fine
  • static debug is helpful in general because people do build against them
  • i recommend we drop cy2019 in the spirit of current year + 3.
  • 2020 clang 7.8 python 3.7.3 C++14 - I am not aware of any platform we support where clang 7 is a requirement, recommend dropping

@xlietz
Copy link
Contributor Author

xlietz commented Jul 2, 2023

Thanks @meshula !

Per comment from Nick and @lgritz , I have updated the OpenEXR Linux build matrix as follows:

2023 gcc 11.2.1 C++17 - Shared Release
2023 gcc 11.2.1 C++17 - Shared Release, Threads OFF
2023 gcc 11.2.1 C++17 - Shared Debug
2023 gcc 11.2.1 C++17 - Static Release
2023 gcc 11.2.1 C++14 - Shared Release
2023 clang 15.0 python 3.10.9 C++17 - Shared Release
2023 clang 14.0 python 3.10.9 C++17 - Shared Release
2022 gcc 9.3.1 C++17 - Shared Release
2021 gcc 9.3.1 C++17 - Shared Release
2020 gcc 6.3.1 C++14 - Shared Release

@xlietz xlietz requested a review from meshula July 2, 2023 02:35
Signed-off-by: Christina Tempelaar-Lietz <xlietz@gmail.com>
Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@cary-ilm cary-ilm merged commit f3fc12b into AcademySoftwareFoundation:main Jul 9, 2023
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants