Hi Ian,

I'm resending the patch changed in a way you prefer (as I hope) with comparison of void_p and int sizes. Generally speaking I like the idea you have described in previous patch email thread to better support cross-compiling, but I would still prefer if the patch may be merged in (even temporarily!) just to unbreak builds on Solaris. Once this is done, I'll be able to continue with reseting solaris buildbot zone etc.

Thanks!
Karel



On 11/ 8/12 10:04 PM, Karel Gardas wrote:
Note that PRIdPTR is considered as linux-ism so it's not available on platforms
like Solaris, although some other free Unix(-like) OSes apparently supports
it too.
---
  includes/mkDerivedConstants.c |   10 ++++++++++
  1 file changed, 10 insertions(+)

diff --git a/includes/mkDerivedConstants.c b/includes/mkDerivedConstants.c
index 7009a3f..bd2214e 100644
--- a/includes/mkDerivedConstants.c
+++ b/includes/mkDerivedConstants.c
@@ -30,6 +30,16 @@
  #include<stdio.h>
  #include<string.h>

+#if !defined(PRIdPTR)
+#if SIZEOF_VOID_P == SIZEOF_INT
+/* compiling for 32bit target */
+#define PRIdPTR "d"
+#else
+/* compiling for 64bit target */
+#define PRIdPTR "ld"
+#endif
+#endif
+
  enum Mode { Gen_Haskell_Type, Gen_Haskell_Value, Gen_Haskell_Wrappers, 
Gen_Haskell_Exports, Gen_Header } mode;

  #define str(a,b) #a "_" #b



_______________________________________________
Cvs-ghc mailing list
Cvs-ghc@haskell.org
http://www.haskell.org/mailman/listinfo/cvs-ghc

Reply via email to