#14371: Race condition in singular doctest
------------------------------------+---------------------------------------
       Reporter:  jdemeyer          |         Owner:  mvngu       
           Type:  defect            |        Status:  needs_review
       Priority:  blocker           |     Milestone:  sage-5.9    
      Component:  doctest coverage  |    Resolution:              
       Keywords:                    |   Work issues:              
Report Upstream:  N/A               |     Reviewers:              
        Authors:  Jeroen Demeyer    |     Merged in:              
   Dependencies:                    |      Stopgaps:              
------------------------------------+---------------------------------------

Comment (by vbraun):

 This doesn't make sense, if something is wrong with the file descriptor
 then write would cause EBADF not EINVAL. Are you sure that the error comes
 from writing to the old, dead subprocess and not to the newly-forked one?
 And why are we even catching RuntimeError in sage.interface.singular.set?

 I would guess that the issue is the absence of `forkpty()` on Solaris?
 I.e. no `os.forkpty` in Python, so `pty.fork` falls back to an alternate
 implementation which presumably triggers a race in Solaris. Contact
 upstream ;-)

 It seems ridiculous to change the doctest here instead of adding a
 workaround to `sage.interface`. If this doesn't work then your platform
 just can't run Sage.

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