#9992: Python scripts try to run before Python is built.
------------------------+---------------------------------------------------
   Reporter:  drkirkby  |       Owner:  GeorgSWeber
       Type:  defect    |      Status:  new        
   Priority:  major     |   Milestone:  sage-4.6   
  Component:  build     |    Keywords:             
     Author:            |    Upstream:  N/A        
   Reviewer:            |      Merged:             
Work_issues:            |  
------------------------+---------------------------------------------------

Comment(by leif):

 Replying to [comment:3 drkirkby]:
 > Replying to [comment:1 leif]:
 > >  * It "rewrites" '''every''' file that contains {{{python}}} and
 {{{#!}}} in its first line [...]
 > That's not such a big deal.

 I think it should skip files matching {{{sage-*}}} and those whose first
 line already starts with {{{#!/usr/bin/env ...}}}.

 > > I don't agree moving its functionality into {{{sage-spkg}}} is a good
 idea. (IIRC it is called by other scripts as well, and {{{sage-spkg}}}
 shouldn't be blown up too much.)
 >
 > OK, well this is easy to fix by putting an
 {{{
 #!sh
 if [ -f '$SAGE_LOCAL/bin/python" ] ; then
     # Run the python script.
 fi
 }}}

 That's an improvement, though it is not clear if we in the near future
 make Python a prerequisite (or intentionally use a system Python for such
 if one is present).

 > I'll leave you to modify the python script if you want to.

 I thought you were going to rewrite it as a shell script (which could as
 well be called from {{{sage-spkg}}}).

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