Summary: | [GTK OSX] libwebkit2gtk fails to link due to missing symbol ANGLEPlatformCurrent | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeremy Huddleston Sequoia <jeremyhu> | ||||||||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | achristensen, bugs-noreply, commit-queue, mcatanzaro, mrobinson | ||||||||||
Priority: | P2 | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 126492 | ||||||||||||
Attachments: |
|
Description
Jeremy Huddleston Sequoia
2016-01-15 00:38:24 PST
(In reply to comment #0) > Should the build system be building libANGLE? It looks like it does that > for the Win32 case. It needs to build some of ANGLE on all platforms, but it builds more of it on Windows than it does on Linux and OS X. I think Martin or Alex might know more. Created attachment 269058 [details]
patch
Yep, that did the trick. Patch attached.
Ignore the above. Commented in wrong bug. Should we be adding Platform.cpp to ANGLESupport then? Created attachment 269116 [details]
patch
Patch to add Platform.cpp to ANGLESupport. This allowed the build to succeed for me.
Comment on attachment 269116 [details] patch Rejecting attachment 269116 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 269116, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: -> origin/master Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ... Currently at 195158 = 5e0ec89178c50a38ebca33d1b802e472421349f9 r195159 = 15508cbafa592ada83ca8fc3ac951d3c9c7b68c8 r195160 = b60e2997410b19de712975d1ed475259837d146b Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output: http://webkit-queues.webkit.org/results/696316 Hm, I thought the bot had a nicer error message for this... anyway, the problem is the patch does not have a changelog entry. You can create a changelog entry template by running Tools/Scripts/prepare-ChangeLog. See https://webkit.org/contributing-code/#changelogs for details. Thanks! Created attachment 269156 [details]
Updated patch with ChangeLog entry
Comment on attachment 269156 [details] Updated patch with ChangeLog entry Attachment 269156 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/697700 New failing tests: imported/w3c/web-platform-tests/streams-api/readable-streams/garbage-collection.html Created attachment 269161 [details]
Archive of layout-test-results from ews117 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117 Port: mac-yosemite Platform: Mac OS X 10.10.5
Comment on attachment 269156 [details]
Updated patch with ChangeLog entry
Hm, a flaky test... now you're just getting unlucky!
(Please only set r+ if you're a reviewer; you don't need to set it when uploading a patch that already has the reviewed-by line.)
Comment on attachment 269156 [details] Updated patch with ChangeLog entry Clearing flags on attachment: 269156 Committed r195169: <http://trac.webkit.org/changeset/195169> All reviewed patches have been landed. Closing bug. |