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

[Alexander Strasser]

On 2013-11-29 11:19 -0800, Timothy Gu wrote:
> 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.

  I do not see a real problem still. I do not know of any policy that
.txt files are not allowed in doc/ which would seem rather awkward to
me.

  Then and now I do not think it makes sense to install the fate.txt
file. You do not actually install FFmpeg to make a fate client. It
is meant to be read out of the source directory.

  Alexander
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131129/2bcc3fd7/attachment.asc>