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

[build_usd.py] Update OpenEXR dependency to 3.1.13 #3077

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

nvidia-jomiller
Copy link
Collaborator

Description of Change(s)

Updates OpenEXR dependency from v3.1.11 -> v3.1.13

Fixes Issue(s)

  • I have verified that all unit tests pass with the proposed changes
  • I have submitted a signed Contributor License Agreement

@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-9650

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgovil
Copy link
Contributor

dgovil commented Oct 23, 2024

@meshula As I understand it, this would only affect USD's dependencies that depend on EXR right? Since you have your own EXR reader inside USD?

That said, 3.1.12 fixes a CVE so it would probably be good to update just to protect folks who transitively use the dependency.

@dgovil dgovil added needs review Issue needing input/review by the repo maintainer (Pixar) build Build-related issue/PR labels Oct 23, 2024
@meshula
Copy link
Member

meshula commented Oct 24, 2024

Yes, that's right. Updating OpenEXR won't affect OpenUSD directly.

pixar-oss pushed a commit that referenced this pull request Dec 9, 2024
Updates OpenEXR dependency from v3.1.11 -> v3.1.13
- This was originally part of PR #3077 from @nvidia-jomiller

Removes FindOpenEXR.cmake because OpenEXR now publishes its own config scripts which supersede a Find script.

Fixes #585
Closes #1844
Closes #3077

(Internal change: 2349684)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build-related issue/PR needs review Issue needing input/review by the repo maintainer (Pixar)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants