RESOLVED FIXED 103918
[CMake] Enable building WebKit sources without Tools
https://bugs.webkit.org/show_bug.cgi?id=103918
Summary [CMake] Enable building WebKit sources without Tools
Laszlo Gombos
Reported 2012-12-03 12:42:41 PST
This build configuration could be useful for: - downstream in a productization branch for package maintainers) where Tools directory might not be used/needed - building just WebKit sources without building Tools for faster build times - ENABLE_TOOLS cmake variable was always set to true, which made not much sense - we should allow it to be configurable
Attachments
proposed patch (1.82 KB, patch)
2012-12-03 12:54 PST, Laszlo Gombos
no flags
Laszlo Gombos
Comment 1 2012-12-03 12:54:34 PST
Created attachment 177313 [details] proposed patch Also moved a section of the code so that the TOOLS_DIR variable is initialized before it gets used. As a test I create a directory with only CMakeLists.txt file and the Source directory in it and successfully built the EFL WebKit port from it by invoking cmake directly (instead of using the build-webkit script).
Gyuyoung Kim
Comment 2 2012-12-04 21:17:17 PST
Comment on attachment 177313 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=177313&action=review Looks find on my side. > CMakeLists.txt:-14 > -SET(ENABLE_WEBCORE ON) Should we move existing place to here ?
Gyuyoung Kim
Comment 3 2012-12-04 21:25:08 PST
Comment on attachment 177313 [details] proposed patch typo : find -> fine.
Laszlo Gombos
Comment 4 2012-12-05 10:54:31 PST
(In reply to comment #2) > (From update of attachment 177313 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=177313&action=review > > Looks find on my side. > > > CMakeLists.txt:-14 > > -SET(ENABLE_WEBCORE ON) > > Should we move existing place to here ? This should be done together with a bigger refactoring as a separate patch. I think this variable should not even be exposed at this level and should be moved under Source - see bug 104083.
WebKit Review Bot
Comment 5 2012-12-05 11:00:10 PST
Comment on attachment 177313 [details] proposed patch Clearing flags on attachment: 177313 Committed r136715: <http://trac.webkit.org/changeset/136715>
WebKit Review Bot
Comment 6 2012-12-05 11:00:15 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.