[FFmpeg-devel] [PATCH 4/5] Revert "doc: fate: Support building a plain text version"

Timothy Gu timothygu99 at gmail.com
Fri Nov 29 20:19:13 CET 2013


On Fri, Nov 29, 2013 at 10:54 AM, Alexander Strasser <eclipse7 at gmx.net> wrote:
> On 2013-11-29 10:17 -0800, Timothy Gu wrote:
>> On Fri, Nov 22, 2013 at 8:25 AM, Stefano Sabatini <stefasab at gmail.com> wrote:
>> > On date Thursday 2013-11-21 16:15:24 -0800, Timothy Gu encoded:
>> >> On Nov 10, 2013 3:47 AM, "Stefano Sabatini" <stefasab at gmail.com> wrote:
>> >> >
>> >> > On date Saturday 2013-11-09 15:36:40 -0800, Timothy Gu encoded:
>> >> > > This reverts commit abab44086fa5b7b3c4cd3f5eabc1348977b506b9.
>> >> > >
>> >> > > Conflicts:
>> >> > >       .gitignore
>> >> > >       configure
>> >> > >       doc/Makefile
>> >> > > ---
>> >> > >  .gitignore   | 1 -
>> >> > >  configure    | 2 --
>> >> > >  doc/Makefile | 9 ---------
>> >> > >  3 files changed, 12 deletions(-)
>> >> >
>> >> > I honestly have no idea why this was done in the first place, so I'd
>> >> > like a comment from Alexander and/or Michael.
>> >>
>> >> Ping
>
>   Sorry, I missed a lot of mails. You can really Cc me at least
> when sending the ping message.

I did CC you :)

>
>> > Also if we want to provide txt files, why this is done only for
>> > fate.texi (what's so special about it?).
>
>   The special thing is that the fate documentation *was* a plain text
> file in the first place. I ported/rewrote it as a texi version so we
> can generate HTML and put that online for browsers. Still a text file
> version can be convenient if you don't want to fire up a browser or
> you have ssh'ed into a remote box.

Thing is your original patch generates a useless file that is not
installed and has a better looking alternative. If a person doesn't
want to read HTML, (s)he can read .texi.

>
>> Ping
>
>   I _do honestly *not* understand_ what this patch is about. This
> leaves me puzzled so I am not sure what to do now. Maybe I should
> read some more random mails and hope to get some usable context.

I was cleaning up the .txt files in doc/ because they either should be
in .texi or in some other places (e.g. wiki, doxygen, etc.). Then I
was amazed that we are actually _generating_ files. And git blame
turns up your commit, so I reverted it.

Timothy


More information about the ffmpeg-devel mailing list