RESOLVED FIXED 137605
[EFL] Make it possible to use ninja for EFL build too 
https://bugs.webkit.org/show_bug.cgi?id=137605
Summary [EFL] Make it possible to use ninja for EFL build too
Éva Balázsfalvi
Reported 2014-10-10 04:06:04 PDT
Make it possible to use ninja for EFL build too.
Attachments
Patch (1.34 KB, patch)
2014-10-10 04:08 PDT, Éva Balázsfalvi 		no flags
DetailsFormatted DiffDiff
Patch (1.79 KB, patch)
2014-10-13 02:54 PDT, Éva Balázsfalvi 		no flags
DetailsFormatted DiffDiff
Patch (1.81 KB, patch)
2014-10-13 05:45 PDT, Éva Balázsfalvi 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Éva Balázsfalvi
Comment 1 2014-10-10 04:08:26 PDT
Created attachment 239617 [details] Patch
Gyuyoung Kim
Comment 2 2014-10-10 22:55:29 PDT
Comment on attachment 239617 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239617&action=review r- due to missing ninja-build in install-dependency. > Tools/Scripts/webkitdirs.pm:1763 > + my $willUseNinja = canUseNinja(); Personally I like to use ninja. To use ninja for EFL, we have to add ninja-build to EFL install-dependency as well.
Éva Balázsfalvi
Comment 3 2014-10-13 02:54:28 PDT
Created attachment 239720 [details] Patch
Csaba Osztrogonác
Comment 4 2014-10-13 03:34:20 PDT
Comment on attachment 239720 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239720&action=review > Tools/efl/install-dependencies:91 > + > + apt-get install ninja-build Why don't we add it simple to the first section, similar to g++, cmake, etc. ?
Éva Balázsfalvi
Comment 5 2014-10-13 05:45:33 PDT
Created attachment 239724 [details] Patch
Csaba Osztrogonác
Comment 6 2014-10-14 04:34:36 PDT
Comment on attachment 239724 [details] Patch r=me
WebKit Commit Bot
Comment 7 2014-10-14 05:09:53 PDT
Comment on attachment 239724 [details] Patch Clearing flags on attachment: 239724 Committed r174681: <http://trac.webkit.org/changeset/174681>
WebKit Commit Bot
Comment 8 2014-10-14 05:09:55 PDT
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.