Dennis Ruffer <[EMAIL PROTECTED]>
writes:

> On Feb 25, 2008, at 11:04 AM, Bee wrote:
>> Not only that, the patch has been added to CVS.
>
> And a fix to ds2texi.fs that was causing my other compiler crash.
>
> However, it is still not doing external calls.
>
> Thanks Bernd

Actually, it would be nice to fix some more issues:


Portability: use "=" when "test", not "==":

--- configure.in.orig   2008-02-01 20:13:07.000000000 +0300
+++ configure.in        2008-02-01 20:16:28.000000000 +0300
@@ -510,7 +510,7 @@
        AC_DEFINE_UNQUOTED(DOUBLE_UCELL_TYPE,$ac_cv_int_type_double_ucell,[an 
unsigned integer type that is twice as long as a pointer])
         OPTS=-ll
 else
-        if test "$ac_cv_sizeof_char_p" == 8; then
+        if test "$ac_cv_sizeof_char_p" = 8; then
            OPTS="-ll -noll"
         else
            OPTS=-noll

"make" is not "gmake":

--- engine/Makefile.in.orig     2008-02-25 14:44:12.000000000 +0300
+++ engine/Makefile.in  2008-02-25 14:48:17.000000000 +0300
@@ -243,7 +243,7 @@
                echo timestamp > stamp-h
 
 Makefile: Makefile.in ../config.status ../configure.in
-       cd .. && make engine/Makefile.in
+       cd .. && ${MAKE} engine/Makefile.in
 
 image.i:       ../fi2c.fs ../$(include_fi)
                $(BOOTFORTH) ../fi2c.fs -e "s\" ../$(include_fi)\" fi2c bye" >$@


It would be nice to keep pre-configure, configure and build stages separate,
autogen.sh isn't supposed to run configure:

--- autogen.sh.orig     2008-02-15 23:25:58.000000000 +0300
+++ autogen.sh  2008-02-15 23:28:22.000000000 +0300
@@ -31,14 +31,7 @@
 # generate header files like engine/config.h.in
 
 autoheader &&
-echo timestamp >stamp-h.in
-
-# run configure Script
-
-./configure &&
+echo timestamp >stamp-h.in &&
 
 #a local gforth.fi probably does not work with $GFORTH
-rm -f gforth.fi &&
-
-echo Running: make bootstrap &&
-make bootstrap
+rm -f gforth.fi


-- 
CKOPO BECHA...
   CKOPO CE3OH...


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to