Move wtf/Platform.h from JavaScriptCore to Source/WTF/wtf
Created attachment 131469 [details] Patch
The goal of this patch is to test and see how badly I break the variou EWSes. Once I get Platform.h moved and successfully building, I will try moving a .h and .cpp pair. Once I get all the builds successfully linking with that, then I will announce on webkit-dev and do the final move.
Qt's current setup is very nice, and will make the final full move very easy. It's not clear to me how I can do a partial move (just one header) with Qt's current VPATH setup. I suspect I may have to use something like: ../../WTF/wtf/Platform.h in their file list. :)
Created attachment 131472 [details] Blind stab at making Qt/EFL build
It's actually more likely that I'll have to fix header include paths for Gtk/Qt/EFL/Chromium to include Source/WTF in WebCore, WebKit, etc.
Comment on attachment 131472 [details] Blind stab at making Qt/EFL build Attachment 131472 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11949159
Comment on attachment 131472 [details] Blind stab at making Qt/EFL build Attachment 131472 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11947258
Comment on attachment 131472 [details] Blind stab at making Qt/EFL build Attachment 131472 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/11942298
Comment on attachment 131472 [details] Blind stab at making Qt/EFL build Attachment 131472 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/11947363
The cr-linux ews bubble will be fixed once bug 81281 lands. I'll look at what's needed to fix qt, win, efl shortly.
Created attachment 132695 [details] Should fix chromium
I'm not going to try and fix Win until bug 80680 lands, as it would involve changing WTF/WTF.vcproj to install into JavaScriptCore/private, which seems wrong...
Created attachment 132700 [details] might fix Qt
If someone from the EFL port could help me understand how to fix EFL before landing, that would be nice.
Created attachment 132714 [details] Should fix EFL and CMake in general
Comment on attachment 132714 [details] Should fix EFL and CMake in general Thanks! (removing review flag for now).
Comment on attachment 132714 [details] Should fix EFL and CMake in general Attachment 132714 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/11991117
Odd. Somehow we broke qt-wk2 in all this.
Or Qt's dependency tracking could just be broken. :) Thus meaning the previous "pass" was likely bogus: ===== BUILD FAILED ====== The build failed with exit code 2. This may have been because you - added an #include to a source/header - added a Q_OBJECT macro to a class - added a new resource to a qrc file as dependencies are not automatically re-computed for local developer builds. You may try computing dependencies manually by running 'make qmake' in: /storage/WebKit-qt-wk2-ews/WebKitBuild/Release or passing --makeargs="qmake" to build-webkit.
Comment on attachment 132714 [details] Should fix EFL and CMake in general Attachment 132714 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/11989489
Comment on attachment 132714 [details] Should fix EFL and CMake in general Attachment 132714 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11986595
Created attachment 132742 [details] fix Mac
Comment on attachment 132742 [details] fix Mac Attachment 132742 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/11999085
Comment on attachment 132742 [details] fix Mac Attachment 132742 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12002072
Comment on attachment 132742 [details] fix Mac Attachment 132742 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11994115
Created attachment 132763 [details] attempt to fix Qt
I'm not even sure that my changes from "Platform.h" to <wtf/Platform.h> are even correct for Qt, since maybe it includes Platform.h in its API? Also, it's not clear to me which .pri/.pro file I would need to edit to add Source/WTF to the header include path for QtWebKit. Ossy?
Comment on attachment 132763 [details] attempt to fix Qt Attachment 132763 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12002103
Created attachment 132771 [details] attempt to fix qt-wk2
Comment on attachment 132771 [details] attempt to fix qt-wk2 Attachment 132771 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/11995134
Comment on attachment 132771 [details] attempt to fix qt-wk2 Attachment 132771 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12000135
Created attachment 132775 [details] Another attempt to fix qt-wk2
I'm going to check the Qt WK2 build today.
I think I'm pretty close. :) The only build I haven't attempted yet is Win. Theoretically EFL works after Raphael's patch update. If you find qt* doesn't work, feel encouraged to upload an updated patch!
Comment on attachment 132775 [details] Another attempt to fix qt-wk2 Attachment 132775 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12002153
Comment on attachment 132775 [details] Another attempt to fix qt-wk2 Attachment 132775 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12007138
Created attachment 132779 [details] FixEfl
Comment on attachment 132779 [details] FixEfl Attachment 132779 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12000165
Comment on attachment 132779 [details] FixEfl Attachment 132779 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12000173
Created attachment 132789 [details] Fix Qt build based on attachment-132779 It isn't the best fix, but it works at least. :)
Tor Arne, could you review/optimize the Qt part of this patch.
Comment on attachment 132789 [details] Fix Qt build based on attachment-132779 Attachment 132789 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12005198
Comment on attachment 132789 [details] Fix Qt build based on attachment-132779 Attachment 132789 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/11995225
Comment on attachment 132789 [details] Fix Qt build based on attachment-132779 Attachment 132789 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12007232
Comment on attachment 132779 [details] FixEfl Clearing r? flag from the patch.
Sweet! thank you both for your help!
Created attachment 132854 [details] Attempt to fix Win-EWS
Comment on attachment 132854 [details] Attempt to fix Win-EWS Attachment 132854 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12022003
win-ews hates me. :p Just gonna wait for bug 80680 to land, or for someone with windows-fu to fix the patch.
Created attachment 132910 [details] Attempt to fix win-ews again
Comment on attachment 132910 [details] Attempt to fix win-ews again Attachment 132910 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12043019
Created attachment 132912 [details] Attempt to fix win-ews again
Comment on attachment 132912 [details] Attempt to fix win-ews again Attachment 132912 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12070030
Created attachment 132922 [details] another attempt to fix win-ews
Created attachment 132934 [details] Patch
A soon as the EWSes roll green, this is ready for landing. (They should all be green now.) Will send an announcement email to webkit-dev about the move.
Comment on attachment 132934 [details] Patch All green!! They said it couldn't be done.
Comment on attachment 132934 [details] Patch Rejecting attachment 132934 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: geLog Hunk #1 succeeded at 1 with fuzz 3. patching file Tools/DumpRenderTree/efl/CMakeLists.txt patching file Tools/DumpRenderTree/qt/DumpRenderTree.pro patching file Tools/QtTestBrowser/QtTestBrowser.pro patching file Tools/Scripts/build-webkit patching file CMakeLists.txt patching file ChangeLog Hunk #1 succeeded at 1 with fuzz 3. Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Adam Barth']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ Full output: http://queues.webkit.org/results/12072181
Committed r111504: <http://trac.webkit.org/changeset/111504>
This broke the Windows build, which I fixed in http://trac.webkit.org/changeset/111550