Skip to content
This repository was archived by the owner on Jan 30, 2023. It is now read-only.

Commit 4061f54

Browse files
committed
Trac #29617: remove failing ECL test.
There is a test in sage/interfaces/tests.py that checks whether or not the "ecl" executable handles stdout and stderr being "full" at the same time. This fails without a custom patch, and therefore with everyone's system copy of ecl. This commit removes the test: running "ecl" on the command-line is not a special sage interface to ecl. This is an upstream ecl bug that will someday get fixed and be added to the ecl test suite. In the meantime, it has no ill effects in sage besides the failing test.
1 parent 444d828 commit 4061f54

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/sage/interfaces/tests.py

+1-3
Original file line numberDiff line numberDiff line change
@@ -23,16 +23,14 @@
2323
Singular
2424
2525
Test that write errors to stderr are handled gracefully by GAP
26-
(see :trac:`13211`) and ECL (see :trac:`14426`) and other interfaces::
26+
(see :trac:`13211`) and other interfaces::
2727
2828
sage: import subprocess
2929
sage: try:
3030
....: f = open('/dev/full', 'w')
3131
....: except IOError:
3232
....: f = open('/dev/null', 'w')
3333
sage: kwds = dict(shell=True, stdout=f, stderr=f)
34-
sage: subprocess.call("echo syntax error | ecl", **kwds) in (0, 255)
35-
True
3634
sage: subprocess.call("echo syntax error | gap", **kwds) in (0, 1)
3735
True
3836
sage: subprocess.call("echo syntax error | gp", **kwds)

0 commit comments

Comments
 (0)