google / sagetv

SageTV is a cross-platform networked DVR and media management system
http://forums.sagetv.com/
Apache License 2.0
265 stars 174 forks source link

Fix conversion to MKV #442

Closed jvl711 closed 3 years ago

jvl711 commented 3 years ago

There was an inadvertent consequence to adding a format substitution from "MATROSKA,WEBM" -> "MATROSKA". This caused conversions to MKV container to break.

For more info see https://forums.sagetv.com/forums/showthread.php?t=66492

Narflex commented 3 years ago

Thanks for the fix. This will break this for the plugin format detector, but it can get fixed within that plugin itself since this breaks things in Sage TV otherwise.

jvl711 commented 3 years ago

@Narflex

Thank you for handling this quickly. I will have an update for the format parser in the next week or so. FYI, I have another pull request coming soon for some enhancements to fixed transcoding. Giving a little more time for some beta testing feedback.

Thanks, Josh