Re: [Maxima-discuss] input stream redirection + SBCL, was: Test bench failure with ask (out of memory with sbcl)

[有坂富雄 <ari...@ic...>]

On 2018-1-2, Robert Dodier <rob...@gm...> wrote:

> On 2017-12-18, 有坂富雄 <ari...@ic...> wrote:
>
>> Thank you for having an interest in my patch.
>> I attached the patch to this mail.
>
> Thank you very much. I have applied the patch as commit 9e32329.
>
> I've also applied the same changes to CMUCL as well as SBCL (commit
> 19aea2d). This enables CMUCL to pass tests/rtest_ask.mac and it makes
> writefile and appendfile work as expected.
>
> Thanks again for your help,
>
> Robert Dodier
>


Thank you for your help!
I'm glad to hear that the patch also works on CMUCL.

By the way,
on Maxima-5.41.0+SBCL, the command line option "--run-string" does not work expectedly.
It seems the interactive mode is not supplied. It is the same as the batch mode.
On current git version of Maxima+SBCL, the command line option "--run-string" does not work expectedly, too.

I think that "--run-string" needs changing the stream.
When the stream made by "make-string-input-stream" reaches the EOF, the stream should be changed to *standard-input*.

So I made a patch to fix this issue.
The patch is based on "maxima-code-bfe48a6d261efd6fa113428941bfcc4c85fcb30d".
The new boolean variable *maxima-run-string* is added.
I attached the patch to this mail.

The result of Maxima compiled with the patch is the following:
----------------------------------------
$ rmaxima --quiet --run-string='build_info(); 10!; factor(%);'
(%i1) build_info()
(%o1)
Maxima version: "5.41post"
Maxima build date: "2018-01-04 02:47:14"
Host type: "x86_64-apple-darwin15.6.0"
Lisp implementation type: "SBCL"
Lisp implementation version: "1.4.3"
(%i2) 10!
(%o2)                               3628800
(%i3) factor(%)
                                   8  4  2
(%o3)                             2  3  5  7
(%i4) :lisp *maxima-run-string*

NIL
(%i4) quit();
$
----------------------------------------

If you have the time, please take a look at the patch.

Thanks,

Tomio Arisaka