[with patch; positive review] singular prompt problem on solaris sparc

[was]

Credit to William Stein, Mike Hansen, and Michael Abshoff?

[was]

I tested it, and it works.

[mabshoff]

Oops, there are three doctest failures with this patch applied:

        sage -t -long devel/sage/sage/interfaces/expect.py # 1 doctests failed
        sage -t -long devel/sage/doc/en/developer/coding_in_other.rst # 2 doctests failed
        sage -t -long devel/sage/doc/en/constructions/algebraic_geometry.rst # 4 doctests failed

Cheers,

Michael

[mabshoff]

This is a 4.0 blocker.

Mike: Any comments?

Cheers,

Michael

[mhansen]

All of the failures are harmless. I've attached a new patch which fixes the doctests.

[mabshoff]

Hmm, I am not so sure the patch does fix every problem: Without the patch:

----------------------------------------------------------------------
| Sage Version 4.0.rc0, Release Date: 2009-05-21                     |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: singular.eval("intvec G = 4,4,4,0,0,0;")
''

With the patch applied:

mabshoff@sage:/scratch/mabshoff/sage-4.0.rc1$ ./sage
----------------------------------------------------------------------
| Sage Version 4.0.rc0, Release Date: 2009-05-21                     |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: singular.eval("intvec G = 4,4,4,0,0,0;")
'intvec G = 4,4,4,0,0,0;'
sage: 

If you look at the attached patch here it seems that we sometimes get the echo of the command and some times not. It might be consistent, i.e. the doctests pass on all platforms (I hope), but something still seems fishy.

I am not saying we shouldn't apply the patch since it fixes a much more severe bug, I just think that there is more to the story. Anyway, doctests do pass, so I will open a followup ticket and merge this patch.

Cheers,

Michael

[mabshoff]

Merged in Sage 4.0.rc1.

Cheers,

Michael