#15184: CachedFunction file location seems problematic
--------------------------+----------------------------
       Reporter:  nbruin  |        Owner:
           Type:  defect  |       Status:  needs_review
       Priority:  major   |    Milestone:  sage-5.12
      Component:  misc    |   Resolution:
       Keywords:          |    Merged in:
        Authors:          |    Reviewers:
Report Upstream:  N/A     |  Work issues:
         Branch:          |       Commit:
   Dependencies:          |     Stopgaps:
--------------------------+----------------------------

Comment (by nbruin):

 I guess it doesn't hurt to have the path normalization in, but it might
 not be necessary for the common prefix checking, at least not at far as
 trailing slashes are concerned:
 {{{
 sage: P="/a/b/c/d/e/f"
 sage: Q="/a/b/c/"
 sage: os.path.commonprefix([P,Q])
 '/a/b/c/'
 sage: Qn=os.path.normpath(Q)
 sage: Qn
 '/a/b/c'
 sage: os.path.commonprefix([P,Qn])
 '/a/b/c'
 sage: os.path.relpath(P,Q)
 'd/e/f'
 sage: os.path.relpath(P,Qn)
 'd/e/f'
 sage: os.path.commonprefix([P,Q]) == Q
 True
 sage: os.path.commonprefix([P,Qn]) == Qn
 True
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/15184#comment:10>
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.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to