Re: [PATCH] define own version of PRIdPTR on platform where its not available

Hello Team,

I'm curious if this patch is kind of controversial so you are reluctant 
to merge it. If you find any issue with it please let me know so I can 
change it as per your suggestion. I've done this patch to un-break GHC 
HEAD build failure on my solaris 11 box.

Thanks!
Karel

On 10/30/12 06:59 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 |   11 +++++++++++
>   1 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/includes/mkDerivedConstants.c b/includes/mkDerivedConstants.c
> index 1393112..953b2b5 100644
> --- a/includes/mkDerivedConstants.c
> +++ b/includes/mkDerivedConstants.c
> @@ -30,6 +30,17 @@
>   #include<stdio.h>
>   #include<string.h>
>
> +
> +#if !defined(PRIdPTR)
> +#if defined(_LP64)
> +/* compiling for 64bit target */
> +#define PRIdPTR "ld"
> +#else
> +/* compiling for 32bit target */
> +#define PRIdPTR "d"
> +#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