#11707: Remove `readlink -n` and `realpath` from $SAGE_ROOT/sage
------------------------+---------------------------------------------------
   Reporter:  jdemeyer  |          Owner:            
       Type:  defect    |         Status:  new       
   Priority:  major     |      Milestone:  sage-4.7.2
  Component:  scripts   |       Keywords:  SAGE_ROOT 
Work_issues:            |       Upstream:  N/A       
   Reviewer:            |         Author:            
     Merged:            |   Dependencies:            
------------------------+---------------------------------------------------

Comment(by leif):

 It is not immediately clear why using `pwd -P`, which fully expands
 symbolic links, and therefore would canonicalize `$SAGE_ROOT` as well,
 should break doctesting.

 Even if it did somehow, this should be addressed in the corresponding
 scripts, `sage-{test,ptest,doctest}`, which are currently under work
 anyway, and have certainly changed since #5852 was opened (or last
 modified).

 Taking the canonical path of both the files to test and `$SAGE_ROOT` (or,
 more precisely, of the current branch of its Sage library), there
 shouldn't be any ambiguity in distinguishing Sage library code from other
 code. We also meanwhile have an option `-force_lib` to `sage -t` and its
 friends.

 Slashs, dashs etc. from paths of files to test should never end up in an
 `import` statement, and I don't think they currently could, as we copy
 source files anyway.

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