#12671: attaching files which have spaces in absolute path name fails on second 
run
------------------------------------------------+---------------------------
       Reporter:  ppurka                        |         Owner:  jason     
           Type:  defect                        |        Status:  needs_work
       Priority:  major                         |     Milestone:  sage-5.1  
      Component:  misc                          |    Resolution:            
       Keywords:  attach load preparser sd40.5  |   Work issues:            
Report Upstream:  N/A                           |     Reviewers:            
        Authors:  Punarbasu Purkayastha         |     Merged in:            
   Dependencies:                                |      Stopgaps:            
------------------------------------------------+---------------------------

Comment (by was):

 ddrake -- can you change the title of the trac ticket to reflect what you
 now plan to accomplish?

 > I strongly disagree with the positive review. The existing code is
 totally crazy -- it distinguishes between
 > strings with and without spaces by running eval() on them and catching
 an exception.

 Thanks for calling my code "totally crazy".  I think you are
 mischaracterizing what the code does.   The point of the eval is not to
 "distinguish between strings with and without spaces".  The eval allows
 for the filename to be determined by valid Python expressions, the
 motivating case being
 {{{
    load DATA+"file.sage"
 }}}
 which is commonly used in the notebook.    It was never meant to be used
 with paths that have spaces.

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