Title: [105001] trunk/Tools
Revision
105001
Author
[email protected]
Date
2012-01-13 15:35:58 -0800 (Fri, 13 Jan 2012)

Log Message

Remove --dry-run support from webkit-patch
https://bugs.webkit.org/show_bug.cgi?id=76300

Reviewed by Dirk Pranke.

This is the rest of the patch from Bug 76300.  I landed it separtely
because main.py had some stray conflict markers that I needed to clean
up first.

* Scripts/webkitpy/test/main.py:
(Tester.parse_args):
(Tester._run_tests):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (105000 => 105001)


--- trunk/Tools/ChangeLog	2012-01-13 23:32:42 UTC (rev 105000)
+++ trunk/Tools/ChangeLog	2012-01-13 23:35:58 UTC (rev 105001)
@@ -1,5 +1,20 @@
 2012-01-13  Adam Barth  <[email protected]>
 
+        Remove --dry-run support from webkit-patch
+        https://bugs.webkit.org/show_bug.cgi?id=76300
+
+        Reviewed by Dirk Pranke.
+
+        This is the rest of the patch from Bug 76300.  I landed it separtely
+        because main.py had some stray conflict markers that I needed to clean
+        up first.
+
+        * Scripts/webkitpy/test/main.py:
+        (Tester.parse_args):
+        (Tester._run_tests):
+
+2012-01-13  Adam Barth  <[email protected]>
+
         test-webkitpy does not run
         https://bugs.webkit.org/show_bug.cgi?id=76318
 

Modified: trunk/Tools/Scripts/webkitpy/test/main.py (105000 => 105001)


--- trunk/Tools/Scripts/webkitpy/test/main.py	2012-01-13 23:32:42 UTC (rev 105000)
+++ trunk/Tools/Scripts/webkitpy/test/main.py	2012-01-13 23:35:58 UTC (rev 105001)
@@ -57,8 +57,6 @@
                           help='run all the tests'),
         parser.add_option('-c', '--coverage', action='', default=False,
                           help='generate code coverage info (requires http://pypi.python.org/pypi/coverage)'),
-        parser.add_option('-n', '--dryrun', action='', default=False,
-                          help='do not actually run the tests'),
         parser.add_option('-q', '--quiet', action='', default=False,
                           help='run quietly (errors, warnings, and progress only)'),
         parser.add_option('-s', '--silent', action='', default=False,
@@ -198,10 +196,6 @@
         return modules
 
     def _run_tests(self, dirs, args):
-        # FIXME: implement a more useful dryrun that recurses into the test suite.
-        if self._options.dryrun:
-            return True
-
         if self._options.coverage:
             try:
                 import coverage
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to