From c2fe43361f876c681a6ab553705fec6d62323c22 Mon Sep 17 00:00:00 2001 From: Martin Herkt Date: Wed, 8 Jan 2014 16:00:40 +0100 Subject: Switch PDF manual generation to rst2pdf This finally gets rid of the LaTeX dependency. We should actually be using docultils directly here, but I didn't do this because of all the potential Python 2/3 breakage. --- DOCS/man/docutils.conf | 19 ------------------- DOCS/man/en/mpv.rst | 5 ----- old-configure | 16 ++++++++-------- old-makefile | 9 +++------ wscript | 6 ++---- wscript_build.py | 20 ++++---------------- 6 files changed, 17 insertions(+), 58 deletions(-) delete mode 100644 DOCS/man/docutils.conf diff --git a/DOCS/man/docutils.conf b/DOCS/man/docutils.conf deleted file mode 100644 index 2e4d0dcf43..0000000000 --- a/DOCS/man/docutils.conf +++ /dev/null @@ -1,19 +0,0 @@ -[latex2e writer] -use-latex-docinfo: yes -use-latex-toc: yes -hyperlink-color: Blue!30!RoyalPurple!70!Black -latex-preamble: \usepackage[usenames,dvipsnames]{xcolor} - \usepackage{fullpage} - \usepackage{graphicx} - \renewcommand{\familydefault}{\sfdefault} - - \makeatletter - \renewcommand{\maketitle}{ - \begin{center} - \vspace*{-1.5em} - \begin{Huge} - \@title - \end{Huge} - \end{center} - } - \makeatother diff --git a/DOCS/man/en/mpv.rst b/DOCS/man/en/mpv.rst index 003a36ed53..4109fdabe4 100644 --- a/DOCS/man/en/mpv.rst +++ b/DOCS/man/en/mpv.rst @@ -10,11 +10,6 @@ a movie player :Manual section: 1 :Manual group: multimedia -.. raw:: latex - - \tableofcontents - \pagebreak - SYNOPSIS ======== diff --git a/old-configure b/old-configure index 763d95b227..b77b1bd8ae 100755 --- a/old-configure +++ b/old-configure @@ -934,19 +934,19 @@ else fi echores "$_build_man" -_rst2latex=rst2latex -if [ -f "$(which rst2latex.py)" ] ; then - _rst2latex=rst2latex.py +_rst2pdf=rst2pdf +if [ -f "$(which rst2pdf.py)" ] ; then + _rst2pdf=rst2pdf.py fi -echocheck "whether to build manual PDFs with rst2latex" -texcheck() { - echo test | $_rst2latex --config=DOCS/man/docutils.conf | pdflatex -halt-on-error -draftmode -output-directory="$mplayer_tmpdir" +echocheck "whether to build manual PDFs with rst2pdf" +pdfcheck() { + echo test | $_rst2pdf -c --repeat-table-rows -o "$mplayer_tmpdir/test.pdf" } if test "$_build_pdf" = auto ; then _build_pdf=no - command_check texcheck && _build_pdf=yes + command_check pdfcheck && _build_pdf=yes else _build_pdf=no fi @@ -3269,7 +3269,7 @@ GETCH = $_getch TIMER = $_timer RST2MAN = $_rst2man BUILD_MAN = $_build_man -RST2LATEX = $_rst2latex +RST2PDF = $_rst2pdf BUILD_PDF = $_build_pdf EXESUF = $_exesuf diff --git a/old-makefile b/old-makefile index 23bb62140a..05f1f7d4a7 100644 --- a/old-makefile +++ b/old-makefile @@ -373,11 +373,8 @@ endif all: $(ALL_TARGETS) -%.tex: %.rst - $(RST2LATEX) --config=DOCS/man/docutils.conf $< $@ - -%.pdf: %.tex - pdflatex -interaction=batchmode -jobname=$(basename $@) $<; pdflatex -interaction=batchmode -jobname=$(basename $@) $< +%.pdf: %.rst + $(RST2PDF) -c --repeat-table-rows $< -o $@ %.1: %.rst $(RST2MAN) $< $@ @@ -536,7 +533,7 @@ clean: -$(RM) $(call ADD_ALL_DIRS,/*.o /*.d /*.a /*.ho /*~) -$(RM) $(call ADD_ALL_DIRS,/*.o /*.a /*.ho /*~) -$(RM) $(call ADD_ALL_EXESUFS,mpv) - -$(RM) $(call ADDSUFFIXES,.pdf .tex .log .aux .out .toc,DOCS/man/*/mpv) + -$(RM) $(call ADDSUFFIXES,.pdf,DOCS/man/*/mpv) -$(RM) DOCS/man/*/mpv.1 -$(RM) version.h -$(RM) input/input_conf.h diff --git a/wscript b/wscript index 143a88a94e..e6e70c23ed 100644 --- a/wscript +++ b/wscript @@ -31,8 +31,7 @@ build_options = [ }, { 'name': '--pdf-build', 'desc': 'pdf manual generation', - 'func': check_ctx_vars('RST2LATEX', 'PDFLATEX'), - 'default': 'disable' + 'func': check_ctx_vars('RST2PDF'), }, { 'name': 'libdl', 'desc': 'dynamic loader', @@ -786,8 +785,7 @@ def configure(ctx): ctx.find_program(pkg_config, var='PKG_CONFIG') ctx.find_program('perl', var='BIN_PERL') ctx.find_program('rst2man', var='RST2MAN', mandatory=False) - ctx.find_program('rst2latex', var='RST2LATEX', mandatory=False) - ctx.find_program('pdflatex', var='PDFLATEX', mandatory=False) + ctx.find_program('rst2pdf', var='RST2PDF', mandatory=False) ctx.find_program(windres, var='WINDRES', mandatory=False) for ident, _, _ in _INSTALL_DIRS_LIST: diff --git a/wscript_build.py b/wscript_build.py index 8b7ca79995..d7b5a3b999 100644 --- a/wscript_build.py +++ b/wscript_build.py @@ -24,26 +24,14 @@ def _build_pdf(ctx): from waflib import TaskGen TaskGen.declare_chain( - name = 'rst2latex', - rule = '${RST2LATEX} ${RST2LATEX_FLAGS} ${SRC} ${TGT}', + name = 'rst2pdf', + rule = '${RST2PDF} ${RST2PDF_FLAGS} ${SRC} -o ${TGT}', ext_in = '.rst', - ext_out = '.tex' ) - - TaskGen.declare_chain( - name = 'pdflatex', - rule = '${PDFLATEX} ${PDFLATEX_FLAGS} ${SRC}; ' * 2, - ext_in = '.tex', ext_out = '.pdf', shell = True ) - ctx.env.RST2LATEX_FLAGS = [ - '--config=' + ctx.srcnode.abspath() + '/DOCS/man/docutils.conf' - ] - - ctx.env.PDFLATEX_FLAGS = [ - '--interaction=batchmode', - '--output-directory=DOCS/man/en/', - '--jobname=mpv' + ctx.env.RST2PDF_FLAGS = [ + '-c --repeat-table-rows' ] ctx(source = 'DOCS/man/en/mpv.rst') -- cgit v1.2.3