[FFmpeg-devel] [PATCH] 5 patches from the Debian packaging

[Timothy Gu]

On Sun, Mar 9, 2014 at 11:42 AM, Andreas Cadhalpun
<andreas.cadhalpun at googlemail.com> wrote:
> On 09.03.2014 01:00, Timothy Gu wrote:
>>
>> I can see that the perl implementation of makeinfo/texi2any (there are
>> 2 implementations: a Perl one and an C one) does have rudimentary init
>> file support [1], and many of the variable we use are valid (e.g.
>> PRE_BODY_CLOSE and EXTRA_HEAD) for the Texinfo Perl module [2]. It's
>> just that the subroutine Texinfo::Config::set_from_init_file is never
>> used.
>>
>> Therefore, IMO adding support for it just requires using the function
>> to parse the init file, but I'm no Perl expert, so ...
>
>
> Apparently makeinfo expects the init file to be a perl script calling
> set_from_init_file.

Ah you found out how it works! Nice!

>
> I tried to convert the t2h.init to t2h.pm (attached) and mostly succeded,
> but I think changing the title is not that easy.

Yes. I found some examples that includes adding custom callback for
subroutines that might be helpful:
http://anonscm.debian.org/gitweb/?p=debian-tex/texinfo.git;a=tree;f=tp/init;hb=HEAD

>
> While doing this, I noticed that the favicon.png is not shipped in the
> release tarball. Why not? Is there a license problem or do you have no
> source for it?

Well, it is an image file so i don't think there should be a "source" for it :)

The file is used for the website (e.g. http://ffmpeg.org/ffmpeg.html),
but I guess it's not needed.

Also yes, the license of the file is not clear. We lost contact with
the author of the logo, Hervé Flores, and it was not explicitly
released under a free license. However he did "provide us with
sufficient rights to use the art in any form: not limited to the web
site, t-shirts, and print media; to refer only to the FFmpeg project
at no cost to us" ( from
http://trac.ffmpeg.org/wiki/SubmitALogo#UsageRightsLicenseInformation).

Another problem with your approach is that @float is not handled
correctly. It is also a good idea to support both texi2html and
makeinfo for now as makeinfo doesn't look as stable as texi2html.

Timothy