nrwt: remove deprecated ORWT compatibility command line flags
Created attachment 178081 [details] Patch
I really appreciate you cleaning all this up.
Comment on attachment 178081 [details] Patch Attachment 178081 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/15187046
Ossy, can you let me know when it's safe to land this w/o breaking the http://build.webkit.sed.hu/ waterfall? I'd update the config if I had any idea how to do that :).
(In reply to comment #4) > Ossy, can you let me know when it's safe to land this w/o breaking the http://build.webkit.sed.hu/ waterfall? I'd update the config if I had any idea how to do that :). I removed the obsolete --no-launch-safari and --use-remote-links-to-tests from the master.cfg of build.webkit.sed.hu. Now the test command is: command = ["perl", "./Tools/Scripts/run-webkit-tests", "--no-new-test-results", "--debug-rwt-logging", "--no-sample-on-timeout", "--results-directory", "layout-test-results", "--builder-name", WithProperties("%(buildername)s"), WithProperties("--%(configuration)s")] Is there anything else I should remove?
(In reply to comment #5) > (In reply to comment #4) > > Ossy, can you let me know when it's safe to land this w/o breaking the http://build.webkit.sed.hu/ waterfall? I'd update the config if I had any idea how to do that :). > > I removed the obsolete --no-launch-safari and --use-remote-links-to-tests > from the master.cfg of build.webkit.sed.hu. Now the test command is: > command = ["perl", "./Tools/Scripts/run-webkit-tests", "--no-new-test-results", "--debug-rwt-logging", "--no-sample-on-timeout", "--results-directory", "layout-test-results", "--builder-name", WithProperties("%(buildername)s"), WithProperties("--%(configuration)s")] > > Is there anything else I should remove? You probably want to add --no-show-results back in to replace --no-launch-safari. Otherwise, this should be all you need.
yes, I noticed I missed to add --no-show-results first. But I managed to do it for the second update. :)
Committed r137517: <http://trac.webkit.org/changeset/137517>