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

Bazel support: Switch to Imath 3.1.12 #1856

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

Vertexwahn
Copy link
Contributor

@Vertexwahn Vertexwahn commented Oct 1, 2024

This PR only affects the Bazel build (not the CMake build)

  • Switch to Imath 3.1.12
  • Removal of copts in favor of .bazelrc (users of OpenEXR may want to use a newer C++ Standard for building OpenEXR, e.g. some projects may need to use C++20 or newer versions - current approach forces users to C++17)

Signed-off-by: Vertexwahn <julian.amann@tum.de>
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

@cary-ilm cary-ilm merged commit 0a0bb45 into AcademySoftwareFoundation:main Oct 2, 2024
31 checks passed
cary-ilm pushed a commit that referenced this pull request Oct 6, 2024
Signed-off-by: Vertexwahn <julian.amann@tum.de>
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.

2 participants