CF: Error in imake/configure settings?

[Darren Benham <gecko@benham.net>]

I have the following error report from someone who's trying to compile
the crossfire client for Linux running on an Alpha.  Here's what he has to say:

I havn't gotten the 0.95.1 version yet so I don't know if this has been
addressed but I still thought I'd pass it up to you guys and see what you had
to say about it.

-----Begin-----
  Hi - Client.tmpl sets CC and CFLAGS as appropriate for Digital Unix
when it's compiling on any alpha architecture. Needless to say, this
is unfortunate for linux/alpha. Setting ForceCC to gcc in the
Imakefile was the obvious fix, but it probably isn't the best
one. Perhaps a check for Linux or gcc before the arch-specific checks
in Client.tmpl?
------End------

When I asked him for a patch he gave me this:

-----Begin-----
  Okay, the diff attached below basically forces CC to be 'gcc -O2'
when compiling under any Linux architecture. If you'd prefer one
specific to the Alpha, let me know, but I suspect that thsi is also
going to bite anyone porting to the other hard-coded architectures (eg
MIPS).

  To be quite honest, I don't know why it manually sets the compiler;
xmkmf can and should be doing all the work. Could you perhaps enquire
as to whether that entire section is necessary? It's trying to
replicate the work that's already done in the X11 config, and,
needless to say, is breaking things. Having it commented out, for
example, would leave people able to force the compiler to whatever
they want by uncommenting it, but still rely on the local, known
working, configuration.

diff -ru crossfire-client-0.94.3.orig/Client.tmpl crossfire-client-0.94.3/Clien
t.tmpl
--- crossfire-client-0.94.3.orig/Client.tmpl    Sat Aug  1 17:13:25 1998
+++ crossfire-client-0.94.3/Client.tmpl Mon Dec 14 09:25:16 1998
@@ -214,6 +214,10 @@
     CC = cc -ansi
 #endif

+#ifdef LinuxArchitecture
+    CC = gcc -O2
+#endif
+
 #else /* ForceCC is set */
   CC = ForceCC
 #endif
------End------
-- 
=========================================================================
* http://benham.net/index.html                                     <><  *
* -------------------- * -----BEGIN GEEK CODE BLOCK----- ---------------*
*    Darren Benham     * Version: 3.1                                   *
*  <gecko@benham.net>  * GCS d+(-) s:+ a29 C++$ UL++>++++ P+++$ L++>++++*
*       KC7YAQ         * E? W+++$ N+(-) o? K- w+++$(--) O M-- V- PS--   *
*   Debian Developer   * PE++ Y++ PGP++ t+ 5 X R+ !tv b++++ DI+++ D++   *
*  <gecko@debian.org>  * G++>G+++ e h+ r* y+                            *
* -------------------- * ------END GEEK CODE BLOCK------ ---------------*
=========================================================================

PGP signature