Bug#547797: [PATCH] webkit build failure on ia64

[Mike Hommey]

Package: webkit
Severity: important

On Tue, Sep 22, 2009 at 10:06:14AM +0100, Colin Watson wrote:
> [CCing Mike directly since I'm not subscribed to pkg-webkit-maintainers
> and so I have a suspicion that my mail might get stuck in some queue
> otherwise. Sorry for any resulting duplicates.]

Interestingly, your mail is not in the moderation queue...

Forwarding this as a bug, this will have better visibility.

> webkit in experimental hasn't built on Debian ia64 yet due to a
> dependency-wait
> (http://experimental.debian.net/new/package.php?p=webkit&suite=experimental)
> but it failed to build on Ubuntu ia64:
> 
>   http://launchpadlibrarian.net/32049689/buildlog_ubuntu-karmic-ia64.webkit_1.1.14-1_FAILEDTOBUILD.txt.gz
> 
> It looks like JSVALUE32_64 isn't appropriate on ia64. Would this patch
> make sense? With this, it at least builds, although I have no
> particularly sensible way to test the resulting binary packages.
> 
> --- webkit-1.1.14/JavaScriptCore/wtf/Platform.h.orig	2009-09-22 08:59:48.000000000 +0000
> +++ webkit-1.1.14/JavaScriptCore/wtf/Platform.h	2009-09-21 14:55:33.000000000 +0000
> @@ -294,6 +294,11 @@
>  #define WTF_PLATFORM_BIG_ENDIAN 1
>  #endif
>  
> +/* PLATFORM(IA64) */
> +#if defined(__ia64__)
> +#define WTF_PLATFORM_IA64 1
> +#endif
> +
>  /* PLATFORM(WINCE) && PLATFORM(QT)
>     We can not determine the endianess at compile time. For
>     Qt for Windows CE the endianess is specified in the
> @@ -616,7 +621,7 @@
>  #endif
>  
>  #if !defined(WTF_USE_JSVALUE64) && !defined(WTF_USE_JSVALUE32) && !defined(WTF_USE_JSVALUE32_64)
> -#if PLATFORM(X86_64) && (PLATFORM(MAC) || PLATFORM(LINUX))
> +#if (PLATFORM(X86_64) && (PLATFORM(MAC) || PLATFORM(LINUX))) || PLATFORM(IA64)
>  #define WTF_USE_JSVALUE64 1
>  #elif PLATFORM(ARM) || PLATFORM(PPC64)
>  #define WTF_USE_JSVALUE32 1
> 
> Thanks,
> 
> -- 
> Colin Watson                                       [cjwatson at ubuntu.com]