Summary: | REGRESSION(CMAKE): [GTK] WebKit2 C API tests are not longer built in the same binary | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bunhere, commit-queue, gustavo, gyuyoung.kim, mrobinson, pnormand, rakuco, sergio | ||||
Priority: | P2 | Keywords: | Gtk, Regression | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 132686 | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2014-05-08 09:11:40 PDT
It's kind of nice that we can share the build instructions with other ports, so we should weigh the options carefully. (In reply to comment #1) > It's kind of nice that we can share the build instructions with other ports, so we should weigh the options carefully. But we are building the WTF tests as a single binary, I don't understand why this inconsistency. I'm not sure what we are sharing for the WebKit2 test, because we have a list of binaries in our PlatformGTK file. Created attachment 231144 [details] Patch This would also fix bug #132684, since the single binary is TestWebKit2 Comment on attachment 231144 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=231144&action=review > Tools/TestWebKitAPI/PlatformGTK.cmake:107 > +add_test(TestWebKit2 ${TESTWEBKITAPI_RUNTIME_OUTPUT_DIRECTORY}/WebKit2/TestWebKit2) > +set_tests_properties(TestWebKit2 PROPERTIES TIMEOUT 60) > +set_target_properties(TestWebKit2 PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${TESTWEBKITAPI_RUNTIME_OUTPUT_DIRECTORY}/WebKit2) We don't actually need these lines since we don't use the CMake's 'make test.' (In reply to comment #4) > (From update of attachment 231144 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=231144&action=review > > > Tools/TestWebKitAPI/PlatformGTK.cmake:107 > > +add_test(TestWebKit2 ${TESTWEBKITAPI_RUNTIME_OUTPUT_DIRECTORY}/WebKit2/TestWebKit2) > > +set_tests_properties(TestWebKit2 PROPERTIES TIMEOUT 60) > > +set_target_properties(TestWebKit2 PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${TESTWEBKITAPI_RUNTIME_OUTPUT_DIRECTORY}/WebKit2) > > We don't actually need these lines since we don't use the CMake's 'make test.' I copy pasted like a monkey :-P I guess it doesn't hurt in any case and it's consistent with the other tests Committed r168574: <http://trac.webkit.org/changeset/168574> |