On 4/3/24 02:08, Waldek Hebisch wrote:
On Tue, Apr 02, 2024 at 07:51:53PM +0800, Qian Yun wrote:
It is caused by commit 68a3e8ee in 2023-Feb-9, the removal of
"makePathname" in "histInputFileName".

So the fix is the following.  Please review it before I do the
commit.

AFAICS the PNAME part is not needed: make_filename0 converts
symbols to strings.

At first glance, it looks that change below would break
')history )save' and ')undo'.   I think that we need to change
'historySpad2Cmd' to call an intermediate function to
convert arguments.

Yes, you are right about both points.

I believe the following is enough:

diff --git a/src/interp/i-syscmd.boot b/src/interp/i-syscmd.boot
index 8e337d59..31d2a1fe 100644
--- a/src/interp/i-syscmd.boot
+++ b/src/interp/i-syscmd.boot
@@ -1367,7 +1367,7 @@ historySpad2Cmd() ==
     opt = 'show    => showHistory optargs
     opt = 'change  => changeHistListLen first optargs
     opt = 'restore => restoreHistory optargs
-    opt = 'write   => writeInputLines(optargs,1)
+    opt = 'write   => writeInputLines(first optargs, 1)
   'done



- Qian

--
You received this message because you are subscribed to the Google Groups "FriCAS - 
computer algebra system" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to fricas-devel+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/fricas-devel/9ddd97ee-2693-4ce8-a773-d0d523f09f3c%40gmail.com.

Reply via email to