[GTK] check target looks broken
authormcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Apr 2018 01:07:57 +0000 (01:07 +0000)
committermcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Apr 2018 01:07:57 +0000 (01:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184407

Reviewed by Carlos Garcia Campos.

It's pretty clear that nobody has been using check for a long time.

We got rid of check-for-webkitdom-api-breaks a long time ago, when we stopped autogenerating
the DOM API, but it's still used in the check target. Fix this.

Also, the check target for some reason runs itself from the Source subdirectory of the
builddir, but the test script expects to be run from the toplevel build directory. Fix this
too.

Add VERBATIM for good measure.

* Source/PlatformGTK.cmake:

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

ChangeLog
Source/PlatformGTK.cmake

index b150dfa..e70fb38 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,23 @@
+2018-04-23  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        [GTK] check target looks broken
+        https://bugs.webkit.org/show_bug.cgi?id=184407
+
+        Reviewed by Carlos Garcia Campos.
+
+        It's pretty clear that nobody has been using check for a long time.
+
+        We got rid of check-for-webkitdom-api-breaks a long time ago, when we stopped autogenerating
+        the DOM API, but it's still used in the check target. Fix this.
+
+        Also, the check target for some reason runs itself from the Source subdirectory of the
+        builddir, but the test script expects to be run from the toplevel build directory. Fix this
+        too.
+
+        Add VERBATIM for good measure.
+
+        * Source/PlatformGTK.cmake:
+
 2018-04-20  Daniel Bates  <dabates@apple.com>
 
         Unreviewed, rolling out r230117.
index 6dd6b43..b8527ca 100644 (file)
@@ -53,7 +53,8 @@ endif ()
 
 add_custom_target(check
     COMMAND ${TOOLS_DIR}/Scripts/run-gtk-tests
-    COMMAND ${TOOLS_DIR}/gtk/check-for-webkitdom-api-breaks
+    WORKING_DIRECTORY "${CMAKE_BINARY_DIR}"
+    VERBATIM
 )
 
 if (DEVELOPER_MODE)