Skip to content

Bump to OpenJPH 0.21.0 #1978

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

Merged

Conversation

palemieux
Copy link

Replace my OpenJPH feature branch with OpenJPH 0.21.0

@cary-ilm
Copy link
Member

cary-ilm commented Feb 9, 2025

@palemieux and/or @peterhillman, any idea why the fuzzing workflow fails? It appears to be failing to link with undefined references to openjph and add odd error just before:

/usr/bin/ld: /usr/bin/ld: DWARF error: invalid or unhandled FORM value: 0x25

Does the fuzzer link line need to be updated?

Now that I think about, I believe this workflow was failing on the PR prior to merge, but I assumed it was because of other issues.

Also, @peterhillman, do we need to update something to get the fuzzer to run on the htj2k-beta branch? Presumably, it should.

@palemieux
Copy link
Author

The OpenJPH library might be missing from the oss-fuzz build file for OpenEXR at https://github.com/google/oss-fuzz/blob/bfa03c999c5bc5e69e28d656531a8f5705eb2730/projects/openexr/build.sh#L24 ?

Who has been handling oss-fuzz within the OpenEXR community. I can work with them to fix oss-fuzz.

@cary-ilm
Copy link
Member

cary-ilm commented Feb 9, 2025

We'll resolve the fuzz failure separately, unrelated to this.

@cary-ilm cary-ilm merged commit 813c7cb into AcademySoftwareFoundation:htj2k-beta Feb 9, 2025
31 of 33 checks passed
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