#12631: Get rid of spkg/base/dir-0.1-install
------------------------+---------------------------------------------------
   Reporter:  jdemeyer  |          Owner:  GeorgSWeber   
       Type:  defect    |         Status:  new           
   Priority:  major     |      Milestone:  sage-5.0      
  Component:  build     |       Keywords:                
Work_issues:            |       Upstream:  N/A           
   Reviewer:            |         Author:  Jeroen Demeyer
     Merged:            |   Dependencies:  #12102        
------------------------+---------------------------------------------------

Comment(by jhpalmieri):

 It would be nice not to create `$SAGE_ROOT/tmp` until needed. It would
 require modifying a script or two: `sage-apply-ticket` (and possibly
 `sage-maketest`, but I don't think so). The following might be all that's
 required:
 {{{
 #!diff
 diff --git a/sage-apply-ticket b/sage-apply-ticket
 --- a/sage-apply-ticket
 +++ b/sage-apply-ticket
 @@ -525,7 +525,13 @@ def apply_and_test(n, patches, directory

  def archive_log(ticketnum, tmp_file):
      dir = os.getcwd()
 -    os.chdir(SAGE_ROOT+'/tmp')
 +    tmpdir = os.path.join(SAGE_ROOT, 'tmp')
 +    try:
 +        os.makedirs(tmpdir)
 +    except OSError:
 +        if not os.path.isdir(tmpdir):
 +            raise
 +    os.chdir(tmpdir)
      os.system('cp %s %s-mergelog'%(tmp_file,ticketnum))
      os.system('tar -rf mergelog.tar %s-mergelog'%(ticketnum))
      os.system('rm %s-mergelog'%(ticketnum))
 }}}
 But this could certainly go on another ticket.

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