#5771: [with patch, positive review] %latex should issue a warning if latex
isn't
installed on the system
------------------------+---------------------------------------------------
Reporter: mabshoff | Owner: jhpalmieri
Type: defect | Status: assigned
Priority: major | Milestone: sage-4.0.2
Component: misc | Keywords:
Reviewer: Rob Beezer | Author: John Palmieri
Merged: |
------------------------+---------------------------------------------------
Comment(by rbeezer):
I was thinking today about the doctest failure with the global variable
being set in the "wrong" order. We've seen that movie before with this
file, no? :-;
Latex patch applies and builds against 4.0.1, now passes randomized
testing on sage/misc/latex.py, documentation builds properly.
I think the current doctests are fine. My thought for the two that failed
with missing latex and missing pdflatex were along the lines of the
following (but I haven't tested this at all).
{{{
sage: h, s = have_pdflatex(), png(ZZ[x], "zz.png", do_in_background=False)
sage: (h and s == None) or (not(h) and s.find('Error: PDFLatex') == 0)
True
}}}
That'd cover both possibilities, which was not really possible before the
two new "have_xxx" checks. So consider that simply a suggestion for the
next time you are sprucing up the doctests in this file.
Positive review.
Release manager: Apply #6089 before the "trac_5771.patch" patch.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5771#comment:5>
Sage <http://sagemath.org/>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
