#5994: singular.version() yields an error when first called, has no doctest, and
has a strange output imo
--------------------------+-------------------------------------------------
   Reporter:  SimonKing   |       Owner:  was             
       Type:  defect      |      Status:  needs_info      
   Priority:  major       |   Milestone:  sage-4.3.1      
  Component:  interfaces  |    Keywords:  singular version
     Author:              |    Upstream:  N/A             
   Reviewer:              |      Merged:                  
Work_issues:              |  
--------------------------+-------------------------------------------------
Changes (by SimonKing):

  * status:  needs_review => needs_info


Comment:

 OK, meanwhile I built sage-4.3.1.rc1

 The patch applies cleanly.

 However, I don't like the doc tests, and I think the return value is
 wrong.

 The tests check that the first version number is 3. OK, it will eventually
 change, but not in the near future.

 Then they test that the version number is of length 3. Can we rely on it?
 There used to be two-digit versions.

 In fact, the "official" version number seems to be four digits, not three:
 {{{
 sage: singular.eval('system("version")')
 '3104'
 }}}

 Hence, the first return value of singular.version() should be (3,1,0,4)
 not (3,1,0).
 Note that according to the Singular homepage there is a version 3-1-0-6
 available, so, the last digit does play a role.

 So, my questions are:

  * How important is it to have doc tests that remain valid if the Singular
 version changes?
  * Do we take the patch level version number of Singular serious?

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