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

Remove usage of the Theora codec #3617

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

padenot
Copy link
Contributor

@padenot padenot commented Feb 2, 2024

This fixes #3616.

I haven't touched anything in conformance-suites/, this looked like versioned and frozen test suite, but let me know if I should remove Theora resources there as well.

@CLAassistant
Copy link

CLAassistant commented Feb 2, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@kenrussell kenrussell left a comment

Choose a reason for hiding this comment

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

This looks perfect, thank you! Yes, we should avoid touching the conformance-suites/ snapshots since they're supposed to represent points in time.

@kenrussell
Copy link
Member

Also @padenot could you click through the CLA to satisfy that check? (Mozilla's already a Khronos member, so this doesn't change anything except clearing up that automated check)

@padenot
Copy link
Contributor Author

padenot commented Feb 5, 2024

Also @padenot could you click through the CLA to satisfy that check? (Mozilla's already a Khronos member, so this doesn't change anything except clearing up that automated check)

Done, thanks!

@kenrussell
Copy link
Member

Thank you!

@kenrussell kenrussell merged commit 8f01dfe into KhronosGroup:main Feb 8, 2024
2 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.

Remove usage of the Theora codec
3 participants