#18908: Fix math-readline script
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  dimpase                |       Status:  needs_review
           Type:         |    Milestone:  sage-6.8
  defect                 |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:  Nathann Cohen, Sébastien Labbé,
  scripts                |  Salvatore Stella
       Keywords:         |  Work issues:
        Authors:         |       Commit:
  Jeroen Demeyer         |  a3c75ab48e7a0ef0b5f5c38cf769b58e0be35ba4
Report Upstream:  N/A    |     Stopgaps:
         Branch:         |
  public/18908           |
   Dependencies:         |
-------------------------+-------------------------------------------------

Comment (by jdemeyer):

 Replying to [comment:34 dimpase]:
 > saying that something is incorrect/wrong without giving an explanation
 could create a tension you don't need.

 But it was really incorrect: the fact that the script hangs when `math` is
 not installed was a ''symptom'' of the problem, not the problem itself.

 But at the time when I send that message, I didn't have a clear
 explanation of the real problem: I know that the proposed solution was
 wrong, but I couldn't yet say what the right fix would be. Like I said:
 it's often easier to explain ''after'' you have a working patch.

 And I had to act quickly because, these days, tickets which have
 positive_review are often closed very soon.

--
Ticket URL: <http://trac.sagemath.org/ticket/18908#comment:35>
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.
For more options, visit https://groups.google.com/d/optout.

Reply via email to