Opened 16 years ago
Closed 15 years ago
#19346 closed update (fixed)
webkit-gtk out of date
Reported by: | lee@… | Owned by: | myschizobuddy@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.7.1 |
Keywords: | Cc: | jeremyhu (Jeremy Huddleston Sequoia), dbevans (David B. Evans), vinc17@… | |
Port: | webkit-gtk |
Description
I saw that libXt was holding you up from using a more recent version of webkit. It looks like it builds now, so is there any chance of us getting a more recent version of webkit-gtk? It would be great, because the svn repo is about 1.5 gigs to check out.
Thanks
Attachments (3)
Change History (15)
comment:1 Changed 16 years ago by mf2k (Frank Schima)
Milestone: | Port Bugs → Port Updates |
---|---|
Owner: | changed from macports-tickets@… to myschizobuddy@… |
Port: | webkit-gtk added |
comment:2 Changed 16 years ago by dbevans (David B. Evans)
Cc: | jeremyhu@… added |
---|
Changed 16 years ago by dbevans (David B. Evans)
Attachment: | patch-webkit-gtk.diff added |
---|
Draft patch for Portfile (work in progress)
comment:4 Changed 16 years ago by jeremyhu (Jeremy Huddleston Sequoia)
Welcome to namespace conflict hell... we workaround this in X11 like this:
http://cgit.freedesktop.org/xorg/xserver/tree/hw/xquartz/sanitizedCocoa.h
http://cgit.freedesktop.org/xorg/xserver/tree/hw/xquartz/sanitizedCarbon.h
comment:5 Changed 16 years ago by mtalexander (Mike Alexander)
I wanted to build GnuCash with the new code to use Webkit for report formatting. This required a newer version of Webkit so I created a portfile to build one (before I found this ticket). I think the conflict mentioned above is really a bug in the configure file. It is assuming that it must use the WebCore icu library on Darwin even if pkg_config knows about another one. Assuming I'm correct in my assumption about the icu library, this probably should be fixed upstream. The patch I'll attach also updates libsoup although that may have already been taken care of.
Changed 16 years ago by mtalexander (Mike Alexander)
Attachment: | webkit.patch added |
---|
Update webkit-gtk and libsoup
comment:6 Changed 16 years ago by mtalexander (Mike Alexander)
Oops, I had two patches to webkit and got them confused. The icu library patch is correct, but it doesn't solve the problem with CoreFoundation. That's a different patch to a couple of files (JavaScriptCore/wtf/unicode/icu/CollatorICU.cpp and WebCore/platform/FileSystem.h). They should be checking the PLATFORM(CF) flag before including CoreFoundation headers. Apparently webkit wasn't tested on a non-CF Darwin build. I'll attach that patch too.
Changed 16 years ago by mtalexander (Mike Alexander)
Attachment: | corefoundation.patch added |
---|
Patch to not include CoreFoundation headers if PLATFORM(CF) is false
comment:7 Changed 16 years ago by jmroot (Joshua Root)
Type: | enhancement → update |
---|
comment:10 Changed 16 years ago by vinc17@…
Any news? Liferea 1.6.0 (that should be released later this month) needs a more recent version of webkit and I can't test the release candidate.
comment:12 Changed 15 years ago by dbevans (David B. Evans)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Updated to release version 1.1.7 in r51453.
Yes, but the latest nightly build (svn 42583) won't build yet because it requires
Getting by this, the build still fails because of a conflict in typedefs (for Boolean) between X11 and the Carbon framework.
Not sure whether this is something to fix in webkit-gtk upstream or in X11 but investigating.
Attached is my current draft of a patch to update the webkit-gtk Portfile (last problem still not addressed).