Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 427042 - net-libs/webkit-gtk-1.8.1-r301 ABI=x32 - (?fails to build?)
Summary: net-libs/webkit-gtk-1.8.1-r301 ABI=x32 - (?fails to build?)
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL: https://bugs.webkit.org/show_bug.cgi?...
Whiteboard:
Keywords: PATCH, UPSTREAM
Depends on:
Blocks: x32
  Show dependency tree
 
Reported: 2012-07-18 03:15 UTC by devsk
Modified: 2013-11-18 20:11 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
jsvalue patch (webkit-gtk-1.8.1-jsvalue32.patch,464 bytes, patch)
2012-07-18 03:16 UTC, devsk
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description devsk 2012-07-18 03:15:44 UTC
Its the same failure as for qt-webkit. The attached patch fixes the issue.

Reproducible: Always
Comment 1 devsk 2012-07-18 03:16:36 UTC
Created attachment 318502 [details, diff]
jsvalue patch

Makes webkit-gtk package build without jit USE flag.
Comment 2 Pacho Ramos gentoo-dev 2012-07-21 12:08:11 UTC
Send this to upstream please and post link here:
https://bugs.webkit.org/

Thanks
Comment 3 Alexandre Rostovtsev (RETIRED) gentoo-dev 2012-09-13 01:52:00 UTC
I am not convinced that this patch is a good solution.

First, instead of "defined(__LP64__)", you should use "!defined(__ILP32__)". The __LP64__ check happens to work with gcc on linux, but is not guaranteed to be portable.

Second, perhaps a better solution is to add a new value for the CPU() macro, e.g. WTF_CPU_X32, and use that instead of X86_64. Or maybe make x32 part of CPU(X86)?

So please open a bug with upstream - https://bugs.webkit.org/ - and see what they think. (I am asking *you* to open an upstream bug because you are using x32, and I am not, so you will be able to test things for upstream developers, and I won't.) And as Pacho said, then post the link to the upstream bug here.
Comment 4 piotr5 2012-12-23 08:27:38 UTC
I can confirm this.
https://bugs.webkit.org/show_bug.cgi?id=99153 contains patches to handle this bug. but I guess they are "rebased" for newer versions and therefore not working out of the box...
Comment 5 Pacho Ramos gentoo-dev 2013-08-24 13:24:43 UTC
We should probably wait for upstream, otherwise will be impossible to update patches on every major release
Comment 6 Pacho Ramos gentoo-dev 2013-11-18 20:11:23 UTC
Looks like upstream won't support it for now
https://bugs.webkit.org/show_bug.cgi?id=100450#c17