#5852: [with patch, needs work] the detection of SAGE_ROOT in $SAGE_ROOT/sage
script should expand symlinks recursively (fix this on systems that support
readlink -f)
-----------------------------+----------------------------------------------
    Reporter:  tornaria      |         Owner:  tornaria  
        Type:  defect        |        Status:  needs_work
    Priority:  major         |     Milestone:  sage-4.7.2
   Component:  distribution  |    Resolution:            
    Keywords:                |   Work_issues:            
    Upstream:  N/A           |      Reviewer:            
      Author:                |        Merged:            
Dependencies:                |  
-----------------------------+----------------------------------------------

Comment(by jhpalmieri):

 Has the issue with `sage-doctest` been resolved?  The code now says
 {{{
 #!python
         dev_path = os.path.realpath(os.path.join(SAGE_ROOT, 'devel'))
         our_path = os.path.realpath(argv[1])

         if not force_lib and (ext in ['.spyx', '.sage'] or
                               not dev_path in our_path):
             library_code = False
 }}}
 Since `os.path.realpath` is used twice, shouldn't this be okay?  If not,
 another option is to use
 [http://docs.python.org/library/os.path.html#os.path.samefile
 os.path.samefile].

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5852#comment:15>
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 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.

Reply via email to