It would be ideal if buildbot slaves could upload their built frameworks. This would make it simpler to do a binary search across SVN revisions to track down which revision caused a regression. If possible, the frameworks should be bundled with a launcher similar to the nightly build one so that environment variables don't have to be manually set.
Created attachment 7513 [details] Script + WebKit.app launcher application This patch contains a rewritten version of the nightly build script I have been using for the last few months, as well as the source for the latest WebKit.app launcher. The script has been modified so that it is suitable to be used by the build slaves to provide both per-revision builds as this bug discusses and the nightly builds for http://nightly.webkit.org/.
Created attachment 7514 [details] Binary file missing from the previous patch
Created attachment 7515 [details] Updated patch to fix copy-n-paste error
Comment on attachment 7515 [details] Updated patch to fix copy-n-paste error A couple small comments: 1. No need for a prefix header. 2. the directory for NightlyLauncher should probably be moved up into WebKitTools directly and called NightlyLauncher instead of nightly-launcher. I think it makes sense to follow the form of all the other build scripts and have a "build-nightlylauncher" script separate from upload-webkit-diskimage... Regardless, this is *absolutely phenomenal* and it's great to finally see it moving into svn. Having this all in a central location will be great for the project. I think at least xenon, and probably darin should have a look at this. Xenon because he deals so much with this sort of thing for us internally, and darin because he'll undoubtedly have an opinion about the perl. ;)
Created attachment 7517 [details] Updated patch I've split the build script into two pieces now, and renamed nightly-launcher to WebKitLauncher living directly in the WebKitTools as suggested. build-launcher-app builds WebKit.app and bundles the frameworks inside it. build-launcher-dmg builds a disk image containing WebKit.app. It takes optional arguments to instruct it to upload the disk image as a nightly build, or to an arbitrary location for use by the build slaves.
Created attachment 7524 [details] Further cleanup A few updates after feedback from Eric on IRC. getCurrentSVNRevision has moved into webkitdirs.pm as currentSVNRevision, and it is used in the three locations where its functionality was duplicated. In addition, error messages have been tidied up to provide more information about the failure. WebKitLauncher/main.m has been updated to stick to Cocoa APIs rather than the mix of Cocoa and CoreFoundation it was previously. It should now also meet the coding style guidelines.
Comment on attachment 7524 [details] Further cleanup Looks fine to land. r=me
Landed.
Just a note: I have disabled this from running on the build slaves as the build master is very nearly out of disk space and these disk images seem to be nearly unused.