[FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

[Aman Gupta]

On Wed, Apr 25, 2018 at 9:17 PM, Aman Gupta <ffmpeg at tmm1.net> wrote:

>
> On Wed, Apr 25, 2018 at 6:15 PM Marton Balint <cus at passwd.hu> wrote:
>
>>
>>
>> On Thu, 26 Apr 2018, Carl Eugen Hoyos wrote:
>>
>> > 2018-04-25 4:07 GMT+02:00, Aman Gupta <ffmpeg at tmm1.net>:
>> >
>> >> Processes only teletext pages marked as subtitles, so
>> >> depending on the stream it might not produce any output.
>> >
>> > Shouldn't there be at least an option to output whatever
>> > page was requested?
>> >
>> > +1 for not needing an external dependency.
>>
>> Why? Libzvbi is LGPL. We might as well include *that* in the ffmpeg
>> codebase...
>
>
> I was under the impression that libzvbi was GPL2 like the rest of zapping.
> If I had known it was LGPL I probably wouldn't have gone down this route.
>

Another reason I discounted libzvbi is because it seems unmaintained,
without a new release is over 5 years. It is also not readily available on
macOS via homebrew because of this reason, so I wasn't easily able to try
it out (https://github.com/Homebrew/legacy-homebrew/pull/26315).


>
> It might make sense to merge this back into the zvbi based decoder, but
> I've never used zvbi so I'm not sure how it compares to the features I've
> implemented here. I will try to find some time to investigate, but to be
> honest this version is working very well for my needs and I'm not
> particularly motivated to add a dependency to my project.
>
> Aman
>
>
>>
>> I thought having a decoder which is inferior in features to an existing
>> one or re-inventing yet something again in ffmpeg code are two things we
>> are trying to avoid.
>>
>> Regards,
>> Marton
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>