[FFmpeg-devel] [PATCH 1/2] doc: do not generate doc/avoptions_(codecs|formats).texi
Michael Niedermayer
michaelni at gmx.at
Tue Sep 10 18:52:54 CEST 2013
On Mon, Sep 09, 2013 at 09:56:08AM +0200, Stefano Sabatini wrote:
> On date Sunday 2013-09-08 15:36:54 -0700, Timothy Gu encoded:
> > The contents of the two files have been hardcoded to codecs.texi and
> > formats.texi, so there is no use for generating the two files during build
> > time.
> >
> > Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
> > ---
> > After this patch is applied, the print_options utility is unused. Whether to
> > delete the utility is a RfC.
> > ---
> >
> > .gitignore | 2 --
> > doc/Makefile | 7 -------
> > 2 files changed, 9 deletions(-)
> >
> > diff --git a/.gitignore b/.gitignore
> > index 1f13ec4..807ecb1 100644
> > --- a/.gitignore
> > +++ b/.gitignore
> > @@ -33,8 +33,6 @@
> > /doc/*.html
> > /doc/*.pod
> > /doc/config.texi
> > -/doc/avoptions_codec.texi
> > -/doc/avoptions_format.texi
> > /doc/examples/decoding_encoding
> > /doc/examples/demuxing
> > /doc/examples/filtering_audio
> > diff --git a/doc/Makefile b/doc/Makefile
> > index 7415899..cfd2ca5 100644
> > --- a/doc/Makefile
> > +++ b/doc/Makefile
> > @@ -50,13 +50,6 @@ doc/%.txt: doc/%.texi
> > $(Q)$(TEXIDEP)
> > $(M)makeinfo --force --no-headers -o $@ $< 2>/dev/null
> >
> > -GENTEXI = format codec
> > -GENTEXI := $(GENTEXI:%=doc/avoptions_%.texi)
> > -
> > -$(GENTEXI): TAG = GENTEXI
> > -$(GENTEXI): doc/avoptions_%.texi: doc/print_options$(HOSTEXESUF)
> > - $(M)doc/print_options $* > $@
> > -
> > doc/%.html: TAG = HTML
> > doc/%.html: doc/%.texi $(SRC_PATH)/doc/t2h.init $(GENTEXI)
> > $(Q)$(TEXIDEP)
>
> I suppose it's OK. I suggest to keep the generating program for
> easing merges.
if the programs are kept, code to build and test them should be kept
too.
Also are our handwritten files complete and uptodate compared to the
autobuild ones ?
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130910/2548c573/attachment.asc>
More information about the ffmpeg-devel
mailing list