building HTML version of French tutorial is broken

[sagetrac-mvngu]

Ticket #8036 fixes some issues with non-ASCII characters in the reference manual. But it breaks the building of the HTML version of the French tutorial:

sphinx-build -b html -d /scratch/mvngu/release/sage-4.3.2.alpha1/devel/sage/doc/output/doctrees/fr/tutorial    /scratch/mvngu/release/sage-4.3.2.alpha1/devel/sage/doc/fr/tutorial /scratch/mvngu/release/sage-4.3.2.alpha1/devel/sage/doc/output/html/fr/tutorial
Running Sphinx v0.6.3

Exception occurred:
  File "/scratch/mvngu/release/sage-4.3.2.alpha1/devel/sage/doc/fr/tutorial/conf.py", line 38, in <module>
    latex_preamble += '\\DeclareUnicodeCharacter{00A0}{\\nobreakspace}\n'
NameError: name 'latex_preamble' is not defined
The full traceback has been saved in /tmp/sphinx-err-6XQBIT.log, if you want to report the issue to the author.
Please also report this if it was a user error, so that a better error message can be provided next time.
Send reports to sphinx-dev@googlegroups.com. Thanks!
Build finished.  The built documents can be found in /scratch/mvngu/release/sage-4.3.2.alpha1/devel/sage/doc/output/html/fr/tutorial

This is due to the deletion of the line

latex_preamble = '\usepackage{amsmath}\n\usepackage{amsfonts}\n' 

in trac_8036-docbuild_utf8x.patch. I'm making this a blocker against Sage 4.3.2.

CC: @qed777

Component: documentation

Keywords: French tutorial

Author: John Palmieri

Reviewer: Mitesh Patel

Merged: sage-4.3.2.rc0

Issue created by migration from https://trac.sagemath.org/ticket/8146

[qed777]

comment:1

We need to upgrade SAGE_DOC/fr/tutorial/conf.py to use latex_elements['preamble'] instead of latex_preamble, which is deprecated. I'll check the other documents.

[jhpalmieri]

comment:2

If I either delete the last few lines of SAGE_DOC/fr/tutorial/conf.py -- the ones dealing with latex_preamble -- or if I replace them with lines using latex_elements['preamble'] instead, the build seems to go through either way. I suppose the second option is closer to what we have now, so we should use that? Here's a patch.

[jhpalmieri]

comment:3

Attachment: trac_8146-french.patch.gz

(I didn't check conf.py for the other documents, but they build with just the usual warnings, so I don't think we need to modify them.)

[qed777]

Reviewer: Mitesh Patel

[qed777]

Author: John Palmieri

[sagetrac-mvngu]

Merged: sage-4.3.2.rc0