RESOLVED FIXED 161854
[CMake] Refactor GENERATE_BINDINGS 
https://bugs.webkit.org/show_bug.cgi?id=161854
Summary [CMake] Refactor GENERATE_BINDINGS
Fujii Hironori
Reported 2016-09-11 20:10:24 PDT
[CMake] Refator GENERATE_BINDINGS I want to do some refatoring on GENERATE_BINDINGS for the preparation of Bug 161433. * Use CMakeParseArguments for argument parsing * Use function instread of macro for its own variable scope * Wrap both preprocess-idls.pl and generate-bindings.pl scripts * Donwcase a local variable COMMON_GENERATOR_DEPENDENCIES
Attachments
Patch (17.72 KB, patch)
2016-09-11 20:30 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (17.54 KB, patch)
2016-09-11 21:46 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (17.26 KB, patch)
2016-09-12 01:10 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (17.25 KB, patch)
2016-09-12 02:53 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 2016-09-11 20:30:23 PDT
Created attachment 288546 [details] Patch
Fujii Hironori
Comment 2 2016-09-11 21:46:23 PDT
Created attachment 288548 [details] Patch
Alex Christensen
Comment 3 2016-09-12 00:01:55 PDT
Comment on attachment 288548 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=288548&action=review > Tools/DumpRenderTree/CMakeLists.txt:129 > + PREFIX JS Could we get rid of this now that the only generated bindings are JS?
Fujii Hironori
Comment 4 2016-09-12 00:47:49 PDT
Thank you for reviewing my patch. (In reply to comment #3) > Could we get rid of this now that the only generated bindings are JS? Yes. I'll remove PREFIX and EXTENSION.
Gyuyoung Kim
Comment 5 2016-09-12 00:48:54 PDT
Comment on attachment 288548 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=288548&action=review > Source/WebCore/CMakeLists.txt:3626 > + GENERATOR JS One question, will you use other generator(e.g. v8) in future ?
Fujii Hironori
Comment 6 2016-09-12 01:10:02 PDT
Created attachment 288558 [details] Patch
Fujii Hironori
Comment 7 2016-09-12 01:18:19 PDT
(In reply to comment #5) > > Source/WebCore/CMakeLists.txt:3626 > > + GENERATOR JS > > One question, will you use other generator(e.g. v8) in future ? No, I don't. This 'GENERATOR' argument takes 'JS', 'TestRunner' or 'DumpRenderTree' now.
Fujii Hironori
Comment 8 2016-09-12 02:53:18 PDT
Created attachment 288562 [details] Patch Fix typos.
Gyuyoung Kim
Comment 9 2016-09-12 17:37:38 PDT
Comment on attachment 288562 [details] Patch Patch looks good to me. r=me. However it would be good if someone might to want to have final review before landing.
Fujii Hironori
Comment 10 2016-09-15 03:14:07 PDT
Thank you for r+. Could anyone give cq+?
WebKit Commit Bot
Comment 11 2016-09-15 10:50:13 PDT
Comment on attachment 288562 [details] Patch Clearing flags on attachment: 288562 Committed r205982: <http://trac.webkit.org/changeset/205982>
WebKit Commit Bot
Comment 12 2016-09-15 10:50:19 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.