#6495: Build the reference manual incrementally
---------------------------------------------------------------+------------
       Reporter:  mpatel                                       |         Owner: 
 tba         
           Type:  enhancement                                  |        Status: 
 needs_review
       Priority:  major                                        |     Milestone: 
 sage-pending
      Component:  documentation                                |    Resolution: 
             
       Keywords:  days38                                       |   Work issues: 
             
Report Upstream:  N/A                                          |     Reviewers: 
 Volker Braun
        Authors:  Mitesh Patel, John Palmieri, Florent Hivert  |     Merged in: 
             
   Dependencies:  #13143                                       |      Stopgaps: 
             
---------------------------------------------------------------+------------

Comment (by jhpalmieri):

 [attachment:trac_6495-redirect_html.patch] doesn't quite work, because
 `os.path.split(path)` always returns a list of length 2. With this change,
 it works for me:
 {{{
 #!diff
 diff --git a/doc/common/builder.py b/doc/common/builder.py
 --- a/doc/common/builder.py
 +++ b/doc/common/builder.py
 @@ -391,7 +391,7 @@ class WebsiteBuilder(DocBuilder):
                      redirect_filename = os.path.join(reference_dir,
 shorter_path, filename)

                      # the number of levels up we need to use in the
 relative url
 -                    levels_up = len(os.path.split(shorter_path))
 +                    levels_up = len(shorter_path.split(os.sep))

                      # the relative url that we will redirect to
                      redirect_url = "/".join(['..']*levels_up +
 [document_name, shorter_path, filename])
 }}}
 Fix that, and fix the docstring for the html method, and I think this part
 is good.

 The warning is still an issue, and I don't know why it still shows up
 during the second pass for the reference manual when doing `sage
 --docbuild all html` but not when running `sage --docbuild reference html`
 twice. Florent, any ideas? Does anyone else see this?

 kcrisman, I don't know about doctests for all of these methods.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6495#comment:165>
Sage <http://www.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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to