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/pkgs: Remove libtheora #39175

Merged
merged 1 commit into from
Jan 4, 2025
Merged

Conversation

gmou3
Copy link
Contributor

@gmou3 gmou3 commented Dec 20, 2024

It is an experimental package with unclear usefulness. The version is from 2013.

There were discussions to remove it back in 2017: https://groups.google.com/g/sage-devel/c/olOxh1f6-cc.

Copy link

Documentation preview for this PR (built with commit a93f139; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@kwankyu
Copy link
Collaborator

kwankyu commented Dec 21, 2024

For record, it was added in #7297.

According to https://en.wikipedia.org/wiki/Theora, the theora video format now seems to have only historical significance.

I agree with its removal from sage.

vbraun pushed a commit to vbraun/sage that referenced this pull request Dec 23, 2024
    
It is an experimental package with unclear usefulness. The version is
from 2013.

There were discussions to remove it back in 2017:
https://groups.google.com/g/sage-devel/c/olOxh1f6-cc.
    
URL: sagemath#39175
Reported by: gmou3
Reviewer(s): Kwankyu Lee
vbraun pushed a commit to vbraun/sage that referenced this pull request Jan 1, 2025
    
It is an experimental package with unclear usefulness. The version is
from 2013.

There were discussions to remove it back in 2017:
https://groups.google.com/g/sage-devel/c/olOxh1f6-cc.
    
URL: sagemath#39175
Reported by: gmou3
Reviewer(s): Kwankyu Lee
@vbraun vbraun merged commit 1981422 into sagemath:develop Jan 4, 2025
30 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants