[clisp-list] Re: compiling clisp-2.31 on FreeBSD-5.2

[Sam S. <sd...@gn...>]

> * Sam Halliday <sbzzvy@lnubb.vr> [2003-12-17 16:48:42 +0000]:
>
> Sam Steingold wrote:
>> > i was trying to compile clisp-2.31 on a FreeBSD-5.2-RC1
>> > system, when the compile failed with this error:
>> > sed: -e expression #1, char 88: Unknown option to `s'
>> I suspect that this is the same as
>> <http://sourceforge.net/tracker/?group_id=1355&atid=101355&func=detail&aid=648079> 
>> could you please investigate?
>
> yes, i belive it is the same bug. the attached patch (using #s instead
> of ,s for delimiters) seems to fix the compilation problems.

this is not a solution. '#' might be rare, but it is just as legal as ','.
moreover, you kept the line quoting commas:

-      LIBS=`echo $LIBS | sed s/','/'\\,'/g`
+      LIBS=`echo $LIBS | sed 's#','#'\\,'#g'`

this does not make any sense - why quote commas if they will not have
any special meaning?

however, please try either of the following two patches, in turn - i.e.,
apply first, try, _remove_ it, apply second, try again, and tell me if
either helps.
thanks.

> however, the following test seems to fail:
>
> Form: (LET ((H (MAKE-HASH-TABLE :TEST `(,(LAMBDA (A B) (LIST (LIST '= A B)) (= A
> B)) . ,(LAMBDA (X) (LET ((Z (SXHASH (COERCE X 'DOUBLE-FLOAT)))) (LIST `((HASH
> ,X) => ,Z)) Z)))))) (LOOP :FOR I :FROM 0 :TO 1000 :DO (SETF (GETHASH I H)
> (FORMAT NIL "~r" I))) (LOOP :FOR I :FROM 0 :TO 1000 :UNLESS (STRING= (GETHASH
> (FLOAT I 1.0d0) H) (GETHASH (FLOAT I 1.0s0) H)) :COLLECT I))
> CORRECT: NIL
> CLISP  : (892)

I think this has been fixed in the CVS.
could you please check that?

> FreeBSD-5.3-RC1 uses a prerelease of gcc-3.3.3

you might want to remove -fomit-frame-pointer from the CFLAGS
if the CVS still exhibits this bug.

PS. please keep the discussion on the list.  thanks.

-- 
Sam Steingold (http://www.podval.org/~sds) running w2k
<http://www.camera.org> <http://www.iris.org.il> <http://www.memri.org/>
<http://www.mideasttruth.com/> <http://www.honestreporting.com>
Beauty is only a light switch away.


--- clisp-link.in.~1.15.~	2003-07-31 19:42:07.823688600 -0400
+++ clisp-link.in	2003-12-17 12:00:56.242352000 -0500
@@ -282,7 +282,7 @@
     verbose "$destinationdir"/${LISPRUN} -B "$installbasedir" -M "$lispinitdir"/lispinit.mem -norc -q -i $to_load -x "(saveinitmem \"$destinationdir/lispinit.mem\")"
     # Generate new makevars
     echo "LIBS: before: "$LIBS
-    LIBS=`echo $LIBS | sed s/','/'\\\\,'/g`
+    LIBS=`echo $LIBS | sed s/','/'\\,'/g`
     echo "LIBS: after: "$LIBS
     sed -e "s,^LIBS=.*\$,LIBS='${LIBS}'," -e "s,^FILES=.*\$,FILES='${FILES}'," < "$sourcedir"/makevars > "$destinationdir"/makevars
     # Done.





--- clisp-link.in.~1.15.~	2003-07-31 19:42:07.823688600 -0400
+++ clisp-link.in	2003-12-17 12:01:25.604572800 -0500
@@ -422,7 +422,7 @@
         verbose "$destinationdir"/${LISPRUN} -B "$installbasedir" -M "$sourcedir"/lispinit.mem -norc -q -i ${LOAD} -x "(saveinitmem \"$destinationdir/lispinit.mem\")"
       fi
       # Generate new makevars
-      LIBS=`echo $LIBS | sed s/','/'\\,'/g`
+      LIBS=`echo $LIBS | sed s/','/'\\\\,'/g`
       sed -e "s,^LIBS=.*\$,LIBS='${LIBS}'," -e "s,^FILES=.*\$,FILES='${FILES}'," < "$sourcedir"/makevars > "$destinationdir"/makevars
     fi
     # Done.