There should be a way to specify additional repository paths in NRWT
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Oct 2013 21:24:18 +0000 (21:24 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Oct 2013 21:24:18 +0000 (21:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=123135

Reviewed by Alexey Proskuryakov.

Added --additional-repository-name and --additional-repository-path options to NRWT.
These options allow us to submit addtional repository information, such as of Safari's
to the new flakiness dashboard.

* Scripts/webkitpy/layout_tests/run_webkit_tests.py:
(parse_args):
* Scripts/webkitpy/port/base.py:
(Port.repository_paths):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@157815 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
Tools/Scripts/webkitpy/port/base.py

index ebf2614f80e98bbf52bbaae9f2cb9750a5b21780..712f629546e8df26b586213e6014109fe2e4e9a2 100644 (file)
@@ -1,3 +1,19 @@
+2013-10-22  Ryosuke Niwa  <rniwa@webkit.org>
+
+        There should be a way to specify additional repository paths in NRWT
+        https://bugs.webkit.org/show_bug.cgi?id=123135
+
+        Reviewed by Alexey Proskuryakov.
+
+        Added --additional-repository-name and --additional-repository-path options to NRWT.
+        These options allow us to submit addtional repository information, such as of Safari's
+        to the new flakiness dashboard.
+
+        * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
+        (parse_args):
+        * Scripts/webkitpy/port/base.py:
+        (Port.repository_paths):
+
 2013-10-22  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r157774.
 2013-10-22  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r157774.
index dd4e8d8fbb19f6e70252007507806f8797da0caf..fdc20cb392edbbd2aa8a907d64bd6b646342c65f 100644 (file)
@@ -295,6 +295,10 @@ def parse_args(args):
             help=("If specified, upload results json files to this appengine server.")),
         optparse.make_option("--results-server-host", default="",
             help=("If specified, upload results JSON file to this results server.")),
             help=("If specified, upload results json files to this appengine server.")),
         optparse.make_option("--results-server-host", default="",
             help=("If specified, upload results JSON file to this results server.")),
+        optparse.make_option("--additional-repository-name",
+            help=("The name of an additional subversion or git checkout")),
+        optparse.make_option("--additional-repository-path",
+            help=("The path to an additional subversion or git checkout (requires --additional-repository-name)")),
     ]))
 
     option_parser = optparse.OptionParser()
     ]))
 
     option_parser = optparse.OptionParser()
index 7abd3e064536223f410f38e719ba26bfde53acb4..821df7d69f6fdaa935938fd7d8b8e750d641cecd 100644 (file)
@@ -1090,7 +1090,10 @@ class Port(object):
 
         # We use LayoutTest directory here because webkit_base isn't a part of WebKit repository in Chromium port
         # where turnk isn't checked out as a whole.
 
         # We use LayoutTest directory here because webkit_base isn't a part of WebKit repository in Chromium port
         # where turnk isn't checked out as a whole.
-        return [('WebKit', self.layout_tests_dir())]
+        repository_paths = [('WebKit', self.layout_tests_dir())]
+        if self.get_option('additional_repository_name') and self.get_option('additional_repository_path'):
+            repository_paths += [(self._options.additional_repository_name, self.path_from_webkit_base(self._options.additional_repository_path))]
+        return repository_paths
 
     _WDIFF_DEL = '##WDIFF_DEL##'
     _WDIFF_ADD = '##WDIFF_ADD##'
 
     _WDIFF_DEL = '##WDIFF_DEL##'
     _WDIFF_ADD = '##WDIFF_ADD##'