Hi Falk,

I'm reluctant to apply changes that should be applied upstream. Also
note that Alpha is not an officially supported platform for ZeroC Ice.
Therefore there may be more problems than just icecpp not compiling.

Your suggestion requires config.h to include limits.h which is a bad
thing IMO. icecpp is just a renamed cccp from GNU gcc 2.8 and the right
way to handle this issue is through autoconf (as GNU version does).
ZeroC sadly removed autoconf from their build system and tweaked
hand-edited macros to deal with multiple archs.

Besides it is unlikely that ZeroC will accept your patch (I forwarded it
to them just in case) because there are plenty of limits.h out there
without WCHAR_MAX defined.  OTH GNU gcc no longer uses WCHAR in cpp.
Therefore I don't think it would stay there for long.

I do not have an Alpha system, would you be so kind to test if the
following simpler patch works for you?

--- src/icecpp/config.h~        2006-03-20 00:42:36.000000000 +0000
+++ src/icecpp/config.h         2006-03-20 00:47:51.000000000 +0000
@@ -63,7 +63,7 @@
 #if defined(_WIN32)
 #   define WCHAR_TYPE_SIZE 2
 #elif (defined(__linux) || defined(__FreeBSD__)) && \
-      (defined(__i386) || defined(__x86_64) || defined(__sparc)) || \
+      (defined(__i386) || defined(__x86_64) || defined(__sparc) || \
+       defined(__mips) || defined(__alpha__)) || \
        defined (__sun) || defined(__hpux) || defined(__APPLE__) || \
        defined(_AIX) || defined(__osf1__)
 #   define WCHAR_TYPE_SIZE 4



El mié, 22-03-2006 a las 22:57 +0100, Falk Hueffner escribió:
> Package: zeroc-ice
> Severity: important
> Justification: fails to build from source
> 
> zeroc-ice fails to build on Alpha:
> 
> [...]
> make[3]: Entering directory `/tmp/buildd/zeroc-ice-3.0.1/src/icecpp'
> cc -c -I../../include  -O2 -I. -DPREFIX=\"\" cccp.c
> In file included from cccp.c:21:
> config.h:80:5: error: #error "unsupported operating system or platform"
> make[3]: *** [cccp.o] Error 1
> make[3]: Leaving directory `/tmp/buildd/zeroc-ice-3.0.1/src/icecpp'
> 
> Full log at;
> http://buildd.debian.org/fetch.php?&pkg=zeroc-ice&ver=3.0.1-1&arch=alpha&stamp=1142961876&file=log&as=raw
> 
> I'd suggest to replace the logic in config.h with something like
> 
> #if WCHAR_MAX == 32767 || WCHAR_MAX == 65536
> #   define WCHAR_TYPE_SIZE 2
> #elif WCHAR_MAX == 2147483647 || WCHAR_MAX == 4294967295
> #   define WCHAR_TYPE_SIZE 4
> #else
> #   error "unsupported operating system or platform"
> #endif
> 
>       Falk
> 
> 
> -- System Information:
> Debian Release: testing/unstable
>   APT prefers unstable
>   APT policy: (500, 'unstable')
> Architecture: alpha
> Shell:  /bin/sh linked to /bin/bash
> Kernel: Linux 2.6.16-rc4-dirty
> Locale: LANG=C, [EMAIL PROTECTED] (charmap=ISO-8859-15)
> 
> 



Reply via email to