test-webkitpy never rebuilds lldbWebKitTester
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 1 Jul 2018 06:00:27 +0000 (06:00 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 1 Jul 2018 06:00:27 +0000 (06:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187229

Reviewed by David Kilzer.

Have test-webkitpy always call script build-lldbwebkittester to build lldbWebKitTester, if needed.

Currently test-webkitpy only calls script build-lldbwebkittester if lldbWebKitTester does not
exist in the build products directory for the active configuration (e.g. Debug); => we do not
build lldbWebKitTester again if its source changes (say, as a result of hacking on lldbWebKitTester).
Instead we should have test-webkitpy unconditionally call build-lldbwebkittester to ensure
that we have an up-to-date build of lldbWebKitTester.

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

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

Tools/ChangeLog
Tools/Scripts/webkitpy/test/main.py

index c961a76..11e4936 100644 (file)
@@ -1,3 +1,22 @@
+2018-06-30  Daniel Bates  <dabates@apple.com>
+
+        test-webkitpy never rebuilds lldbWebKitTester
+        https://bugs.webkit.org/show_bug.cgi?id=187229
+
+        Reviewed by David Kilzer.
+
+        Have test-webkitpy always call script build-lldbwebkittester to build lldbWebKitTester, if needed.
+
+        Currently test-webkitpy only calls script build-lldbwebkittester if lldbWebKitTester does not
+        exist in the build products directory for the active configuration (e.g. Debug); => we do not
+        build lldbWebKitTester again if its source changes (say, as a result of hacking on lldbWebKitTester).
+        Instead we should have test-webkitpy unconditionally call build-lldbwebkittester to ensure
+        that we have an up-to-date build of lldbWebKitTester.
+
+        * Scripts/webkitpy/test/main.py:
+        (Tester._run_tests):
+        (_find_lldb_webkit_tester): Deleted.
+
 2018-06-30  Michael Catanzaro  <mcatanzaro@igalia.com>
 
         [CMake] build-webkit should be able to enable address sanitizer
index 7237a60..4e0b8e4 100644 (file)
@@ -52,12 +52,6 @@ _host = SystemHost()
 _webkit_root = None
 
 
-def _find_lldb_webkit_tester():
-    config = Config(_host.executive, _host.filesystem)
-    lldb_webkit_tester_executable = os.path.join(config.build_directory(config.default_configuration()), 'lldbWebKitTester')
-    return os.path.isfile(lldb_webkit_tester_executable) and os.access(lldb_webkit_tester_executable, os.X_OK)
-
-
 def _build_lldb_webkit_tester():
     if not _host.platform.is_mac():
         _log.error('lldbWebKitTester is not supported on this platform.')
@@ -194,12 +188,10 @@ class Tester(object):
         autoinstall_everything()
 
         if will_run_lldb_webkit_tests:
-            self.printer.write_update("Checking lldbWebKitTester ...")
-            if not _find_lldb_webkit_tester():
-                self.printer.write_update("Building lldbWebKitTester ...")
-                if not _build_lldb_webkit_tester():
-                    _log.error('Failed to build lldbWebKitTester.')
-                    return False
+            self.printer.write_update('Building lldbWebKitTester ...')
+            if not _build_lldb_webkit_tester():
+                _log.error('Failed to build lldbWebKitTester.')
+                return False
 
         if self._options.coverage:
             _log.warning("Checking code coverage, so running things serially")