-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Fix subtitle encoder if subrip is requested #7960
Fix subtitle encoder if subrip is requested #7960
Conversation
I would at least add it to the consts that already exist |
I thought about this too and didn't do it because technically srt and subrip are equivalent and having both of them in the enum would imply they aren't... |
Can you also take a look at the external ass/ssa subtitle? It doesn’t work for me either. |
Pass-through should already be supported:
I added barebone writers for SSA and ASS too, so in theory any text sub can be converted to both now if client requests it. |
4f6f1ef
to
b60905f
Compare
(cherry picked from commit ae79bbc) Signed-off-by: Joshua Boniface <joshua@boniface.me>
Changes
Properly convert subtitles to subrip if requested.
Issues
Fixes #7958
Fixes jellyfin/Swiftfin#417