Re: [webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
Also note Maciej's proposal about moving it to a build step. I am for this idea, but in that case maybe I want the option to ignore run-bindings-tests in the build. When we change CodeGenerator*.pm, we repeat try error many times until build succeeds. If we needed to update run-bindings-tests for each try, that would be troublesome:) On Mon, Apr 9, 2012 at 9:25 AM, Ryosuke Niwa rn...@webkit.org wrote: Also note Maciej's proposal about moving it to a build step. Either way, it shouldn't be ran as a test step on build bots. - Ryosuke On Mon, Apr 9, 2012 at 9:18 AM, Ryosuke Niwa rn...@webkit.org wrote: We have discussed this before: https://lists.webkit.org/pipermail/webkit-dev/2011-September/017868.html A better change would be for us to generate the diff on EWS, and get rid of binding tests step from build bots since they aren't really testing anything. The primary use case of run-bindings-tests is to see the diff before/after a code generator change and it has no business running on the bot or in check-webkit-style. - Ryosuke On Mon, Apr 9, 2012 at 6:39 AM, Vineet Chaudhary rgf...@motorola.com wrote: Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354 . Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev -- Kentaro Hara, Tokyo, Japan (http://haraken.info) ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
[webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354 . Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. +1 to the idea. I think that it would make sense to warn if any file under WebCore/bindings/ is changed. Another idea might be to run ./run-bindings-tests in ./check-webkit-style (instead of warning), but it is too heavy (it takes 5~ seconds). run-bindings-tests are just for showing reviewers what changes the patch is going to make. Even if run-bindings-tests fail, it is not a serious problem. Thus, the build bots ignore the run-bindings-tests results. Consequently, people sometimes forget to update the results. I've been manually rebaselining run-bindings-tests results more than once a week:) On Mon, Apr 9, 2012 at 3:39 PM, Vineet Chaudhary rgf...@motorola.com wrote: Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354 . Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev -- Kentaro Hara, Tokyo, Japan (http://haraken.info) ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
[Sending from a correct mail address...] IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. +1 to the idea. I think that it would make sense to warn if any file under WebCore/bindings/ is changed. Another idea might be to run ./run-bindings-tests in ./check-webkit-style (instead of warning), but it is too heavy (it takes 5~ seconds). run-bindings-tests are just for showing reviewers what changes the patch is going to make. Even if run-bindings-tests fail, it is not a serious problem. Thus, the build bots ignore the run-bindings-tests results. Consequently, people sometimes forget to update the results. I've been manually rebaselining run-bindings-tests results more than once a week:) On Mon, Apr 9, 2012 at 3:39 PM, Vineet Chaudhary rgf...@motorola.com wrote: Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354 . Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev -- Kentaro Hara, Tokyo, Japan (http://haraken.info) -- Kentaro Hara, Tokyo, Japan (http://haraken.info) ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
We have discussed this before: https://lists.webkit.org/pipermail/webkit-dev/2011-September/017868.html A better change would be for us to generate the diff on EWS, and get rid of binding tests step from build bots since they aren't really testing anything. The primary use case of run-bindings-tests is to see the diff before/after a code generator change and it has no business running on the bot or in check-webkit-style. - Ryosuke On Mon, Apr 9, 2012 at 6:39 AM, Vineet Chaudhary rgf...@motorola.comwrote: Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354 . Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
Also note Maciej's proposal about moving it to a build step. Either way, it shouldn't be ran as a test step on build bots. - Ryosuke On Mon, Apr 9, 2012 at 9:18 AM, Ryosuke Niwa rn...@webkit.org wrote: We have discussed this before: https://lists.webkit.org/pipermail/webkit-dev/2011-September/017868.html A better change would be for us to generate the diff on EWS, and get rid of binding tests step from build bots since they aren't really testing anything. The primary use case of run-bindings-tests is to see the diff before/after a code generator change and it has no business running on the bot or in check-webkit-style. - Ryosuke On Mon, Apr 9, 2012 at 6:39 AM, Vineet Chaudhary rgf...@motorola.comwrote: Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354 . Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] check-webkit-style should remind folks to update the results for run-bindings-tests
[From the correct email] On Mon, Apr 9, 2012 at 12:26 PM, Jarred Nicholls jar...@sencha.com wrote: On Mon, Apr 9, 2012 at 12:18 PM, Ryosuke Niwa rn...@webkit.org wrote: We have discussed this before: https://lists.webkit.org/pipermail/webkit-dev/2011-September/017868.html A better change would be for us to generate the diff on EWS, and get rid of binding tests step from build bots since they aren't really testing anything. The primary use case of run-bindings-tests is to see the diff before/after a code generator change and it has no business running on the bot or in check-webkit-style. Either way, we concluded that running the binding tests is very quick and cheap, so wherever it is still effective to prevent regressions in code generation is fine by me; on an as-needed basis (EWS) is clearly ideal, rather than on all builds. - Ryosuke On Mon, Apr 9, 2012 at 6:39 AM, Vineet Chaudhary rgf...@motorola.comwrote: Hi All, It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests. Many times authors forgot to update these binding results. IMO we should add check in ./check-webkit-style to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified. I have filed bug for this https://bugs.webkit.org/show_bug.cgi?id=83354. Please let me know if any suggestions to make this change. Thanks, Vineet ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev -- *Sencha* Jarred Nicholls, Senior Software Architect @jarrednicholls http://twitter.com/jarrednicholls ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev