[FFmpeg-devel] [PATCH] doc/fate: Move fate config example into doc subdirectory

[Alexander Strasser]

Hi Clément!

Clément Bœsch wrote:
> On Mon, Sep 17, 2012 at 10:53:35PM +0200, Alexander Strasser wrote:
> > Be compatible with texi2html 5.0 which doesn't search include paths
> > beginning with . or .. or / inside the specified search paths (-I).
> > 
> > Signed-off-by: Alexander Strasser <eclipse7 at gmx.net>
> > ---
> > 
> >   I finally found an alternative way to do it while I was in the
> > progress of sending this patch. Change only fate.texi like this:
> > 
> > - at verbatiminclude ../tests/fate_config.sh.template
> > + at verbatiminclude examples/../../tests/fate_config.sh.template
> > 
> >   Pretty hacky though and I do not know if it will work with
> > alternative Texinfo processing tools.
> > 
> >   Please tell me your opinion.
> > 
> >  doc/fate.texi                          | 2 +-
> >  {tests => doc}/fate_config.sh.template | 0
> >  2 files changed, 1 insertion(+), 1 deletion(-)
> >  rename {tests => doc}/fate_config.sh.template (100%)
> > 
> > diff --git a/doc/fate.texi b/doc/fate.texi
> > index e04f62f..0f8b442 100644
> > --- a/doc/fate.texi
> > +++ b/doc/fate.texi
> > @@ -97,7 +97,7 @@ configuration variables can be found at @file{tests/fate_config.sh.template}.
> >  
> >  @ifhtml
> >    The mentioned configuration template is also available here:
> > - at verbatiminclude ../tests/fate_config.sh.template
> > + at verbatiminclude fate_config.sh.template
> 
> I think it's pretty fine with the move to doc. It should be fixed in the
> upstream, but the fate config template is just part of the documentation.
> You can even rename it to doc/fate_config.sh.example if you don't like the
> template naming in the documentation.
> 
> Don't forget to fix the directory references (there is at least one just
> above).

  Done & pushed.

Thanks for the report and the review,
  Alexander