Opened 13 years ago

Closed 13 years ago

#3715 closed defect (fixed)

[with spkg, positive review] Bug in lisp startup script (trivial to do bug fix)

Reported by: was Owned by: mabshoff
Priority: minor Milestone: sage-3.0.6
Component: misc Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description 

While debugging the new release of the FriCAS optional package Waldek
Hebisch discovered a small bug in the script that starts clisp. There
are missing argument quotes that can cause problems if some spkg (such
as fricas-1.0.3.spkg) needs to pass arguments containing spaces.

http://sage.math.washington.edu/home/page/packages/lisp.diff

--- lisp.orig   2008-07-20 17:22:27.000000000 -0400
+++ lisp        2008-07-20 03:04:00.000000000 -0400
@@ -1,2 +1,2 @@
 #!/bin/sh
-"$SAGE_ROOT/local/bin/clisp.bin" -B "$SAGE_ROOT/local/lib/clisp-2.46" $@
+"$SAGE_ROOT/local/bin/clisp.bin" -B "$SAGE_ROOT/local/lib/clisp-2.46" "$@"

-----

Regards,
Bill Page.

Change History (4)

comment:1 Changed 13 years ago by was

  • Summary changed from trivial to do bug fix to Bug in lisp startup script (trivial to do bug fix)

comment:2 Changed 13 years ago by mabshoff

  • Milestone changed from sage-3.1 to sage-3.0.6
  • Summary changed from Bug in lisp startup script (trivial to do bug fix) to [with spkg, needs review] Bug in lisp startup script (trivial to do bug fix)

The spgk at #3712 fixes the issue.

Cheers,

Michael

comment:3 Changed 13 years ago by mabshoff

  • Owner changed from cwitty to mabshoff
  • Status changed from new to assigned
  • Summary changed from [with spkg, needs review] Bug in lisp startup script (trivial to do bug fix) to [with spkg, positive review] Bug in lisp startup script (trivial to do bug fix)

Due to the positive review at #3712 I am changing this ticket to a positive review also.

Cheers,

Michael

comment:4 Changed 13 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from assigned to closed

Merged in Sage 3.0.6.final

Note: See TracTickets for help on using tickets.