Summary: | [GTK] TestWebKitAPI tests are no longer built since the WebKit2 rename | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bugs-noreply, clopez, darin, mcatanzaro | ||||
Priority: | P2 | Keywords: | Regression | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2017-07-25 04:08:13 PDT
Created attachment 316362 [details]
Patch
Comment on attachment 316362 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316362&action=review > Tools/ChangeLog:10 > + ENABLE_WEBKIT2 in the makefile, that was renamed to ENABLE_WEBKIT. Also, all WebKit2 test paths were updated to > + use WebKit dir, while the tests are still in WebKit2 directory. This patch also removes I think the patch is fine for fixing this issue now, But I suggest to clarify on the bug of the rename if not applying the webkit2->webkit and webkit->webkitlegacy renaming also on the directories Tools/TestWebKitAPI/Tests was intentional or an overlook. Committed r219865: <http://trac.webkit.org/changeset/219865> Of course it's an oversight, we have to rename all the directories. (In reply to Michael Catanzaro from comment #4) > Of course it's an oversight, we have to rename all the directories. Agreed. |