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

[Alexander Strasser]

On 2013-11-29 14:49 -0800, Timothy Gu wrote:
> On Fri, Nov 29, 2013 at 12:00 PM, Alexander Strasser <eclipse7 at gmx.net> wrote:
> > 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:
> >> >   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.
> 
> It is allowed, but there is no reason to use it when better alternatives exist.
> 
> * If a person likes HTML, there you go.
> * If a person doesn't like HTML, (s)he can read .texi
> 
> (I said this in my last mail)

  I got that. But why not just have a generated text version from the
same source. The texi format is there to generate lots of representations
from one source.

> >   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.
> 
> I used the argument that it is not installed to say that there is no
> reason to generate it. I didn't mean that it should be installed.

  And my argument is that if it isn't installed, that is no reason to
not generate it.

  So what's up with insisting on reverting stuff that does not do any
harm at all but can be useful. Having the file there also helps when
people search for it in the same location it used to be. The only
drawback from before is that they can only find it there after having
built it or FFmpeg before.

  If there were any specific problems I would consider reverting and
sacrificing the text version. If there are no issues, I cannot but
continue to find this arbitrary.

  Alexander

P.S.
I really appreciate your docs patches so far. So do not take this
argumment in a wrong way. And if removing those few lines means
that much to you. I think we can do it. But to the very least
the commit message must be changed.
-------------- 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/20131130/8d058654/attachment.asc>