Hello!

On Sat, 26 Aug 2006 18:50:40 +0200, Luca Capello wrote:
> ATM the patch is quite dirty, because it uses grep and the BASH
> parameter substitution to assign the value, while the same function
> should be very simple as:
>
>   check_lisp_variable () {
>     test -r /etc/cl-launchrc && . /etc/cl-launchrc
>     test -r "$HOME/.cl-launchrc" && . "$HOME/.cl-launchrc"
>     test -n ${SOFTWARE_SYSTEM} && LISP=${SOFTWARE_SYSTEM}
>   }
>
> The last line is obviously fault, what I'd like to do is:

It was quite simple, just putting two ! (and I'd like to thanks
[EMAIL PROTECTED] for his support).

The attached patch is tested on cl-launch_1.85-1, but it's generated
against cl-launch_1.86 as at [1].

Comments?

Thx, bye,
Gismo / Luca

Footnotes: 
[1] http://fare.tunes.org/files/cl-launch/

--- cl-launch.sh_1.86	2006-08-17 10:55:45.000000000 +0200
+++ cl-launch.sh	2006-08-29 22:19:11.000000000 +0200
@@ -1642,7 +1642,13 @@
   LAUNCH_FORM="(progn${MAYBE_PACKAGE_FORM}${HASH_BANG_FORM}${LAUNCH_FORMS})"
   exec $LISP_BIN $IMAGE_OPT $IMAGE $OPTIONS $EVAL "$LAUNCH_FORM" $ENDARGS "$@"
 }
+check_lisp_variable () {
+  test -r /etc/cl-launchrc && . /etc/cl-launchrc
+  test -r "$HOME/.cl-launchrc" && . "$HOME/.cl-launchrc"
+  test -n ${!SOFTWARE_SYSTEM} && LISP=${!SOFTWARE_SYSTEM}
+}
 launch_self () {
+  check_lisp_variable
   LAUNCH_FORMS="$(load_form "$PROG")"
   try_all_lisps "$@"
 }

Attachment: pgpuEjfWJKhoN.pgp
Description: PGP signature

Reply via email to