RESOLVED FIXED 147228
Use Ninja on Windows 
https://bugs.webkit.org/show_bug.cgi?id=147228
Summary Use Ninja on Windows
Alex Christensen
Reported 2015-07-23 10:47:09 PDT
Ninja is faster than Visual Studio, especially when generating the WebCore bindings.
Attachments
Patch (7.53 KB, patch)
2015-07-23 10:57 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (8.53 KB, patch)
2015-07-23 15:10 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (8.54 KB, patch)
2015-07-27 17:17 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (8.59 KB, patch)
2015-07-27 17:23 PDT, Alex Christensen 		mrobinson: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2015-07-23 10:57:34 PDT
Created attachment 257358 [details] Patch
Sam Weinig
Comment 2 2015-07-23 12:39:25 PDT
(In reply to comment #0) > Ninja is faster than Visual Studio, especially when generating the WebCore > bindings. Do you have clean / incremental numbers?
Alex Christensen
Comment 3 2015-07-23 15:10:51 PDT
Created attachment 257391 [details] Patch
Alex Christensen
Comment 4 2015-07-23 15:12:22 PDT
(In reply to comment #2) > (In reply to comment #0) > > Ninja is faster than Visual Studio, especially when generating the WebCore > > bindings. > > Do you have clean / incremental numbers? I don't yet, but ninja is probably about twice as fast as sequentially generating all the bindings for WebCore, which is what the CMake build does without this. I would estimate this brings the CMake build time on Windows from about an hour to about a half hour.
Alex Christensen
Comment 5 2015-07-23 16:04:13 PDT
Comment on attachment 257391 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=257391&action=review > Source/cmake/OptionsWindows.cmake:30 > + endif endif ()
Alex Christensen
Comment 6 2015-07-27 17:17:15 PDT
Created attachment 257618 [details] Patch
Alex Christensen
Comment 7 2015-07-27 17:23:48 PDT
Created attachment 257619 [details] Patch
Martin Robinson
Comment 8 2015-07-27 17:30:59 PDT
Comment on attachment 257619 [details] Patch Seems like a safe bet.
Alex Christensen
Comment 9 2015-07-27 17:39:50 PDT
http://trac.webkit.org/changeset/187470
Alex Christensen
Comment 10 2015-07-27 18:21:19 PDT
(In reply to comment #2) > (In reply to comment #0) > > Ninja is faster than Visual Studio, especially when generating the WebCore > > bindings. > > Do you have clean / incremental numbers? Today, a clean build with CMake and MSBuild took 66.7 minutes. A clean build with CMake and Ninja took 27.7 minutes. That's just over 2x as fast, mostly because it parallelizes the WebCore generating phase. MSBuild: https://build.webkit.org/builders/WinCairo%2064-Bit%20Release/builds/47680/steps/compile-webkit/logs/stdio Ninja: https://build.webkit.org/builders/WinCairo%2064-Bit%20Release/builds/47698/steps/compile-webkit/logs/stdio
Note You need to log in before you can comment on or make changes to this bug.