Bug 148127

Summary: [CMake] REGRESSION(r188540): WebKitTestRunner is not longer built and all the tests (layout and perf) fail.
Product: WebKit Reporter: Carlos Alberto Lopez Perez <clopez>
Component: WebKitGTKAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Critical CC: achristensen, bfulgham, calvaris, mcatanzaro, ossy, pnormand
Priority: P1    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch mrobinson: review+

Description Carlos Alberto Lopez Perez 2015-08-18 04:48:46 PDT
Since r188540 WebKitTestRunner is not longer built and all the perf and layout tests fail.

$ find WebKitBuild/Release/ -type f -executable
WebKitBuild/Release/CMakeFiles/3.0.2/CMakeDetermineCompilerABI_C.bin
WebKitBuild/Release/CMakeFiles/3.0.2/CMakeDetermineCompilerABI_CXX.bin
WebKitBuild/Release/CMakeFiles/3.0.2/CompilerIdCXX/a.out
WebKitBuild/Release/CMakeFiles/3.0.2/CompilerIdC/a.out
WebKitBuild/Release/bin/WebKitNetworkProcess
WebKitBuild/Release/bin/jsc
WebKitBuild/Release/bin/WebKitDatabaseProcess
WebKitBuild/Release/bin/WebKitPluginProcess
WebKitBuild/Release/bin/WebKitPluginProcess2
WebKitBuild/Release/bin/LLIntOffsetsExtractor
WebKitBuild/Release/bin/WebKitWebProcess
WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18.2.4
WebKitBuild/Release/lib/libwebkit2gtkinjectedbundle.so
WebKitBuild/Release/lib/libgtest.so
WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37.10.0


$ Tools/Scripts/run-webkit-tests --gtk
Using port 'gtk-wk2'
Test configuration: <, x86, release>
Placing test results in /home/gardener/WebKit/WebKitBuild/Release/layout-test-results
Baseline search path: gtk -> wk2 -> generic
Using Release build
Pixel tests disabled
Regular timeout: 6000, slow test timeout: 30000
Command line: /home/gardener/WebKit/Tools/jhbuild/jhbuild-wrapper --gtk run /home/gardener/WebKit/WebKitBuild/Release/bin/WebKitTestRunner -

--lint-test-files warnings:
LayoutTests/platform/gtk/TestExpectations:565 Path does not exist. css3/masking/mask-svg-clipped-fragmentId.html
LayoutTests/platform/gtk/TestExpectations:938 Path does not exist. platform/gtk/editing/pasteboard/middle-button-paste.html
LayoutTests/platform/gtk/TestExpectations:1135 Path does not exist. css3/masking/mask-svg-invalid-fragmentId.html
LayoutTests/platform/gtk/TestExpectations:1163 Duplicate or ambiguous entry lines LayoutTests/platform/gtk/TestExpectations:1161 and LayoutTests/platform/gtk/TestExpectations:1163. media/video-controller-child-rate.html
LayoutTests/platform/gtk/TestExpectations:1738 Path does not exist. platform/gtk/accessibility/object-attributes.html
LayoutTests/platform/gtk/TestExpectations:1739 Path does not exist. platform/gtk/accessibility/title-and-alt.html
LayoutTests/platform/gtk/TestExpectations:2358 Path does not exist. fast/forms/input-auto-fill-button.html

Found 39949 tests; running 35578, skipping 4371.
WebKitTestRunner was not found at /home/gardener/WebKit/WebKitBuild/Release/bin/WebKitTestRunner
Build check failed


$ Tools/Scripts/run-perf-tests --platform gtk --release
WebKitTestRunner was not found at /home/gardener/WebKit/WebKitBuild/Release/bin/WebKitTestRunner
Build not up to date for /home/gardener/WebKit/WebKitBuild/Release/bin/WebKitTestRunner



This affects at least the GTK port, and probably also the EFL one (not checked).


The bots seems to be working because a clean build was still not triggered, so they are using the WebKitTestRunner from r188540-- however the perf bot is already crashing for all tests.
Comment 1 Alex Christensen 2015-08-18 11:41:04 PDT
Created attachment 259284 [details]
Patch
Comment 2 Alex Christensen 2015-08-18 11:41:27 PDT
Woah, my bad.  Does this fix it?
Comment 3 Alex Christensen 2015-08-18 11:54:40 PDT
Created attachment 259285 [details]
Patch
Comment 4 Alex Christensen 2015-08-18 12:09:38 PDT
http://trac.webkit.org/changeset/188589
Comment 5 Carlos Alberto Lopez Perez 2015-08-18 17:11:35 PDT
(In reply to comment #4)
> http://trac.webkit.org/changeset/188589

And this broke the GTK build :(

[12/157] Linking CXX executable bin/WebKitTestRunner
FAILED: : && /usr/lib/ccache/clang++   -std=c++11 -fcolor-diagnostics -Qunused-arguments -O3 -DNDEBUG -fno-exceptions -fno-strict-aliasing -fno-rtti  -L/home/clopez/webkit/webkit/WebKitBuild/DependenciesGTK/Root/lib64  -fuse-ld=gold -Wl,--disable-new-dtags @CMakeFiles/WebKitTestRunner.rsp  -o bin/WebKitTestRunner  && :
Tools/WebKitTestRunner/CMakeFiles/WebKitTestRunner.dir/TestController.cpp.o:/home/clopez/webkit/.ccache/tmp/TestContro.tmp.trinity.14155.ii:function WTR::TestController::ensureViewSupportsOptionsForTest(WTR::TestInvocation const&): error: undefined reference to 'WTR::TestController::updatePlatformSpecificViewOptionsForTest(WTR::ViewOptions&, WTR::TestInvocation const&) const'
Tools/WebKitTestRunner/CMakeFiles/WebKitTestRunner.dir/TestController.cpp.o:/home/clopez/webkit/.ccache/tmp/TestContro.tmp.trinity.14155.ii:function WTR::TestController::viewOptionsForTest(WTR::TestInvocation const&) const: error: undefined reference to 'WTR::TestController::updatePlatformSpecificViewOptionsForTest(WTR::ViewOptions&, WTR::TestInvocation const&) const'
clang: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Build%29/builds/61366/steps/compile-webkit/logs/stdio
Comment 6 Carlos Alberto Lopez Perez 2015-08-18 17:27:36 PDT
Ok....

The build error is unrelated to this CMake issue. 

Your last patch fixed the CMake issue that uncovered the build error caused by r188548 that was previously unnoticed

Reported the build error here: https://bugs.webkit.org/show_bug.cgi?id=148154

Closing this