Make --port a global option and pass the port information to the commit-queue subprocess
Created attachment 70708 [details] Patch
Comment on attachment 70708 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=70708&action=review Relatively disgusting. BUt not all your fault. Please fix the above nits. > WebKitTools/Scripts/webkitpy/tool/commands/queues.py:130 > - self.options = options # FIXME: This code is wrong. Command.options is a list, this assumes an Options element! > + self._options = options Please dont' remove this fixme, this code is still WRONG. > WebKitTools/Scripts/webkitpy/tool/commands/queues_unittest.py:87 > + queue._options = Mock() > + queue._options.iterations = 3 Um. Clearly wrong. > WebKitTools/Scripts/webkitpy/tool/commands/queuestest.py:83 > + def assert_queue_outputs(self, queue, args=None, work_item=None, expected_stdout=None, expected_stderr=None, expected_exceptions=None, options=None, tool=MockTool()): setting tool=MockTool() is wrong here. That will share an instance of MockTool() for all callers!
Committed r69737: <http://trac.webkit.org/changeset/69737>