We should add a remark about the optional --skip-build argument to the usage message of run-javascriptcore-tests.
Created attachment 97379 [details] Patch
Comment on attachment 97379 [details] Patch Is that how the arg is named for run-webkit-tests? I feel like we must have this sort of arg elsewhere.
(In reply to comment #2) > (From update of attachment 97379 [details]) > Is that how the arg is named for run-webkit-tests? I feel like we must have this sort of arg elsewhere. Both old-run-webkit-tests and run-webkit-tests.py have a --no-build option which is analogous to --skip-build. Note, for old-run-webkit-tests this option isn't listed in its usage message. We should add it. For consistency, we should either chose to use --skip-build or --no-build. Note, to change run-javascriptcore-tests to use the latter we'll need to modify the buildbot configuration which references --skip-build.
Created attachment 97456 [details] Patch Rename --skip-build to --no-build to be consistent with the option of the same name in run-webkit-tests. We should follow up and add a build option to old-run-webkit-tests (it only has --no-build, but new-run-webkit-tests also has a --build option). We should also add the --no-build option to the usage message for old-run-webkit-tests.
Comment on attachment 97456 [details] Patch OK. This will require a master restart.
As discussed with Eric Seidel on IRC yesterday (06/16/2011), I'm going to land this in three parts. Part 1 will add the --build/--no-build option to run-javascriptcore-tests. Part 2 will change the buildbot master config. Part 3 will remove the --skip-build option from run-javascriptcore-tests once the master buildbot pick up this configuration.
Committed part 1 in changeset 89136 <http://trac.webkit.org/changeset/89136>. Comitted buildbot master.cfg (part 2) in changeset 89138 <http://trac.webkit.org/changeset/89138>.
Master updated.
It seems unit tests are failing to build after this patch was landed: http://build.webkit.org/builders/SnowLeopard%20Intel%20Debug%20%28Tests%29/builds/713 Running build-api-tests Compiling TestWebKitAPI failed! program finished with exit code 1 elapsedTime=24.968663
(In reply to comment #9) > It seems unit tests are failing to build after this patch was landed: > http://build.webkit.org/builders/SnowLeopard%20Intel%20Debug%20%28Tests%29/builds/713 > > Running build-api-tests > Compiling TestWebKitAPI failed! > program finished with exit code 1 > elapsedTime=24.968663 It's unclear to me why. I mean, we shouldn't be compiling such API tests for GTK. I filed <https://bugs.webkit.org/show_bug.cgi?id=62892> for this.
(In reply to comment #9) > It seems unit tests are failing to build after this patch was landed: > http://build.webkit.org/builders/SnowLeopard%20Intel%20Debug%20%28Tests%29/builds/713 > I misread, this is also failing on the Snow Leopard Debug build machine. In order to roll this out we need to roll out r89138, restart the buildbot master and then roll out r89136. Using a debug build of r89139, I was able to successfully run build-api-tests --debug (which is called by run-api-tests) on Mac OS 10.6.7. For some reason the Snow Leopard Debug bot just calls build-api-tests (without the --debug flag).
(In reply to comment #11) > For some reason the Snow Leopard Debug bot just calls build-api-tests (without the --debug flag). Support for running unit tests was added to the build bot master config as part of bug #62603. Notice, the patch will make the bot always calls run-api-tests (without the --debug argument). Hence, the bot will attempt a release build, but this fails since the bot has a debug build.
(In reply to comment #12) > (In reply to comment #11) > > For some reason the Snow Leopard Debug bot just calls build-api-tests (without the --debug flag). > > Support for running unit tests was added to the build bot master config as part of bug #62603. Notice, the patch will make the bot always calls run-api-tests (without the --debug argument). Hence, the bot will attempt a release build, but this fails since the bot has a debug build. See bug #62918 for more details.
Committed part 3 (removal of --skip-build option) in changeset 89208 <http://trac.webkit.org/changeset/89208>.