There are several compilation units on Windows that have the same target name, differing only in file extension: MiniBrowser.exe and MiniBrowser.dll DumpRenderTree.exe and DumpRenderTree.dll This causes trouble with the debugging symbols, because both "MiniBrowser.exe" and "MiniBrowser.dll" want to produce "MiniBrowser.pdb" for their symbols. This creates a race condition during builds that results in one of the two sets of debugging symbols being lost, which causes problems with producing usable stack traces. To correct these, we will change the DLL build rules to generate "{Name}Lib.DLL", so that the build output will be: MiniBrowser.exe -> MiniBrowser.pdb MiniBrowserLib.dll -> MiniBrowserLib.pdb DumpRenderTree.exe -> DumpRenderTree.pdb DumpRenderTreeLib.exe -> DumpRenderTreeLib.pdb This should resolve these symbol loading issues and help us get usable stack traces.
Created attachment 263123 [details] Patch
Comment on attachment 263123 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263123&action=review > Tools/TestWebKitAPI/PlatformWin.cmake:118 > +set_target_properties(TestWebKitLib PROPERTIES OUTPUT_NAME "TestWebKitLib") This line should no longer be necessary. The only reason it was necessary was because we wanted the dll to have a different name than the target, and that is no longer true. Same elsewhere in the patch.
Comment on attachment 263123 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263123&action=review >> Tools/TestWebKitAPI/PlatformWin.cmake:118 >> +set_target_properties(TestWebKitLib PROPERTIES OUTPUT_NAME "TestWebKitLib") > > This line should no longer be necessary. The only reason it was necessary was because we wanted the dll to have a different name than the target, and that is no longer true. > Same elsewhere in the patch. Excellent! I'll revise the patch.
Committed r191075: <http://trac.webkit.org/changeset/191075>