#10508: Update ATLAS to stable version 3.10.1
-------------------------------------------------------------------------------------------------+
       Reporter:  vbraun                                                        
                 |         Owner:  tbd                                          
                                                      
           Type:  enhancement                                                   
                 |        Status:  needs_review                                 
                                                      
       Priority:  major                                                         
                 |     Milestone:  sage-5.9                                     
                                                      
      Component:  packages: standard                                            
                 |    Resolution:                                               
                                                      
       Keywords:  ATLAS spkg                                                    
                 |   Work issues:                                               
                                                      
Report Upstream:  Reported upstream. No feedback yet.                           
                 |     Reviewers:  Benjamin Jones, Karl-Dieter Crisman, Dmitrii 
Pasechnik, Georg Weber, François Bissey, John Palmieri
        Authors:  Volker Braun, Jeroen Demeyer, Jean-Pierre Flori               
                 |     Merged in:                                               
                                                      
   Dependencies:  #13160, #13395, #13392, #13416, #12994, #9906, #12883, 
#13123, #13415, #14344  |      Stopgaps:                                        
                                                             
-------------------------------------------------------------------------------------------------+

Comment (by jdemeyer):

 There are at least two bugs in the command
 {{{
 status = (rc >> 8) & 0x7f    # see
 http://docs.python.org/library/os.html#os.wait
 }}}

 The first is that `0x7f` should obviously be `0xff` (we need the high byte
 and bytes are still 8 bits). The second is that the low byte is completely
 ignored: if the process got killed by a signal, we want to know that too.

 I think `subprocess.call()` is a good solution since Python recommends it
 anyway over `os.system()` and there is no need to fiddle with the return
 code (signals are returned as negative return codes).

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10508#comment:419>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to