WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
174164
Rename source directory and project files from WebKit2 to WebKit
https://bugs.webkit.org/show_bug.cgi?id=174164
Summary
Rename source directory and project files from WebKit2 to WebKit
Matthew Hanson
Reported
2017-07-05 10:22:32 PDT
The WebKit framework is dubbed "WebKit2" in source code and project files. This bug tracks renaming "WebKit2" to "WebKit" in source directories, project files, and references to those files. My proposal for this work encompasses four distinct tasks: 1. Rename the source directory The proposed rename is: Source/WebKit2/ => Source/WebKit/ 2. Rename relevant Makefiles, tools, configurations, etc... This must be done in conjunction with (1). If done in separate commits, the commits should be contiguous. 3. Rename the Xcode project The proposed rename is: Source/WebKit2/WebKit2.xcodeproj => Source/WebKit/WebKit.xcodeproj 4. Rename the Visual Studio project The proposed rename is: Source/WebKit2/WebKit2.vcxproj => Source/WebKit/WebKit.vcxproj This work can only be done once
https://bugs.webkit.org/show_bug.cgi?id=174162
is completed.
Attachments
Patch for WebKit2 => WebKit rename
(6.20 MB, patch)
2017-07-10 14:05 PDT
,
Matthew Hanson
no flags
Details
Formatted Diff
Diff
Patch for WebKit2 => WebKit rename (v2)
(12.53 MB, patch)
2017-07-11 09:30 PDT
,
Matthew Hanson
no flags
Details
Formatted Diff
Diff
Patch for WebKit2 => WebKit rename (v3)
(6.24 MB, patch)
2017-07-11 10:28 PDT
,
Matthew Hanson
no flags
Details
Formatted Diff
Diff
Patch for WebKit2 => WebKit rename (v4)
(12.58 MB, patch)
2017-07-11 12:40 PDT
,
Matthew Hanson
no flags
Details
Formatted Diff
Diff
Patch for WebKit2 => WebKit rename (v5)
(6.26 MB, patch)
2017-07-11 13:17 PDT
,
Matthew Hanson
no flags
Details
Formatted Diff
Diff
Patch
(7.78 MB, patch)
2017-07-12 23:40 PDT
,
Michael Catanzaro
buildbot
: commit-queue-
Details
Formatted Diff
Diff
Archive of layout-test-results from ews125 for ios-simulator-wk2
(1.18 MB, application/zip)
2017-07-13 06:31 PDT
,
Build Bot
no flags
Details
All changes Source outside of the actual rename
(68.64 KB, patch)
2017-07-13 19:04 PDT
,
Matthew Hanson
mitz: review+
Details
Formatted Diff
Diff
Show Obsolete
(5)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2017-07-05 10:48:09 PDT
<
rdar://problem/33137595
>
Matthew Hanson
Comment 2
2017-07-10 10:53:33 PDT
I've added the Igalia folks to solicit their help in validating the GTK builds + tests.
Matthew Hanson
Comment 3
2017-07-10 14:05:18 PDT
Created
attachment 315021
[details]
Patch for WebKit2 => WebKit rename Also includes the WebKit => WebKitLegacy rename so that the patch applies cleanly.
Build Bot
Comment 4
2017-07-10 15:59:44 PDT
Attachment 315021
[details]
did not pass style-queue: Traceback (most recent call last): File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 84, in <module> main() File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 79, in main WebKitPatch(os.path.abspath(__file__)).main() File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main result = command.check_arguments_and_execute(options, args, self) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute return self.execute(options, args, tool) or 0 File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute self._sequence.run_and_handle_errors(tool, options, state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors self._run(tool, options, state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run step(tool, options).run(state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/steps/checkstyle.py", line 66, in run self._tool.executive.run_and_throw_if_fail(self._tool.deprecated_port().check_webkit_style_command() + args, cwd=self._tool.scm().checkout_root) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 135, in run_and_throw_if_fail exit_code = self._run_command_with_teed_output(args, child_stdout, **kwargs) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 107, in _run_command_with_teed_output **kwargs) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 473, in popen return subprocess.Popen(string_args, **kwargs) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__ errread, errwrite) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child raise child_exception OSError: [Errno 7] Argument list too long If any of these errors are false positives, please file a bug against check-webkit-style.
Matthew Hanson
Comment 5
2017-07-11 09:30:13 PDT
Created
attachment 315116
[details]
Patch for WebKit2 => WebKit rename (v2)
Matthew Hanson
Comment 6
2017-07-11 10:28:05 PDT
Created
attachment 315125
[details]
Patch for WebKit2 => WebKit rename (v3) Update cMake variables and directory names.
Matthew Hanson
Comment 7
2017-07-11 12:40:11 PDT
Created
attachment 315145
[details]
Patch for WebKit2 => WebKit rename (v4) Further cMake fixes.
Matthew Hanson
Comment 8
2017-07-11 13:17:32 PDT
Created
attachment 315152
[details]
Patch for WebKit2 => WebKit rename (v5) Even more GTK fixes
Build Bot
Comment 9
2017-07-11 18:30:45 PDT
Attachment 315152
[details]
did not pass style-queue: Traceback (most recent call last): File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 84, in <module> main() File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 79, in main WebKitPatch(os.path.abspath(__file__)).main() File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main result = command.check_arguments_and_execute(options, args, self) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute return self.execute(options, args, tool) or 0 File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute self._sequence.run_and_handle_errors(tool, options, state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors self._run(tool, options, state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run step(tool, options).run(state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/steps/checkstyle.py", line 66, in run self._tool.executive.run_and_throw_if_fail(self._tool.deprecated_port().check_webkit_style_command() + args, cwd=self._tool.scm().checkout_root) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 135, in run_and_throw_if_fail exit_code = self._run_command_with_teed_output(args, child_stdout, **kwargs) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 107, in _run_command_with_teed_output **kwargs) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 473, in popen return subprocess.Popen(string_args, **kwargs) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__ errread, errwrite) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child raise child_exception OSError: [Errno 7] Argument list too long If any of these errors are false positives, please file a bug against check-webkit-style.
Michael Catanzaro
Comment 10
2017-07-11 20:27:33 PDT
I'll try to get the CMake builds working tomorrow.
Michael Catanzaro
Comment 11
2017-07-12 23:26:08 PDT
(In reply to Michael Catanzaro from
comment #10
)
> I'll try to get the CMake builds working tomorrow.
Done: * Reverted to legacy framework names in Source/CMakeLists.txt and added FIXMEs; it will be easier to rename these separately later * Updated the paths in Source/WebCore/platform/gtk/po/POTFILES.in. * Fixed Source/WebKit/Scripts/generate-forwarding-headers.pl by removing $frameworkDirectoryName That's all that was needed for GTK. Let's try EWS again....
Michael Catanzaro
Comment 12
2017-07-12 23:40:15 PDT
Created
attachment 315336
[details]
Patch I'm having trouble uploading the patch with webkit-patch: Traceback (most recent call last): File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkit-patch", line 84, in <module> main() File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkit-patch", line 79, in main WebKitPatch(os.path.abspath(__file__)).main() File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main result = command.check_arguments_and_execute(options, args, self) File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute return self.execute(options, args, tool) or 0 File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute self._sequence.run_and_handle_errors(tool, options, state) File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors self._run(tool, options, state) File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run step(tool, options).run(state) File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/steps/preparechangelog.py", line 106, in run self._ensure_bug_url(state) File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/tool/steps/preparechangelog.py", line 58, in _ensure_bug_url if not changelog.latest_entry().bug_id(): File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 362, in latest_entry changelog_file = self._filesystem.open_text_file_for_reading(self.path) File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py", line 236, in open_text_file_for_reading return codecs.open(path, 'r', 'utf8', errors) File "/usr/lib64/python2.7/codecs.py", line 896, in open file = __builtin__.open(filename, mode, buffering) IOError: [Errno 2] No such file or directory: u'/home/mcatanzaro/Projects/WebKit/Source/WebKit/cf/ChangeLog' It's not clear to me if this is a problem that actually needs to be fixed. I bet it's a one-time problem that we can ignore (it seems our scripts aren't prepared for ChangeLog files to disappear and reappear in new locations). So I've prepared a git formatted patch instead. I presume EWS will be able to handle it....
Build Bot
Comment 13
2017-07-13 01:36:04 PDT
Attachment 315336
[details]
did not pass style-queue: Traceback (most recent call last): File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 84, in <module> main() File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 79, in main WebKitPatch(os.path.abspath(__file__)).main() File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main result = command.check_arguments_and_execute(options, args, self) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute return self.execute(options, args, tool) or 0 File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute self._sequence.run_and_handle_errors(tool, options, state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors self._run(tool, options, state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run step(tool, options).run(state) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/steps/checkstyle.py", line 66, in run self._tool.executive.run_and_throw_if_fail(self._tool.deprecated_port().check_webkit_style_command() + args, cwd=self._tool.scm().checkout_root) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 135, in run_and_throw_if_fail exit_code = self._run_command_with_teed_output(args, child_stdout, **kwargs) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 107, in _run_command_with_teed_output **kwargs) File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 473, in popen return subprocess.Popen(string_args, **kwargs) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__ errread, errwrite) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child raise child_exception OSError: [Errno 7] Argument list too long If any of these errors are false positives, please file a bug against check-webkit-style.
Michael Catanzaro
Comment 14
2017-07-13 05:43:19 PDT
Well that got GTK and WPE building, but the forwarding headers change broke Mac: /Volumes/Data/EWS/WebKit/Tools/TestWebKitAPI/Tests/WebKit2Cocoa/OpenAndCloseWindow.mm:33:9: fatal error: 'WebKit/WKWindowFeaturesPrivate.h' file not found #import <WebKit/WKWindowFeaturesPrivate.h> ^ 1 error generated. Looks like we're going to have to migrate Tools/TestWebKitAPI/Tests as well.
Build Bot
Comment 15
2017-07-13 06:31:15 PDT
Comment on
attachment 315336
[details]
Patch
Attachment 315336
[details]
did not pass ios-sim-ews (ios-simulator-wk2): Output:
http://webkit-queues.webkit.org/results/4112966
New failing tests: imported/w3c/IndexedDB-private-browsing/idbfactory_open12.html
Build Bot
Comment 16
2017-07-13 06:31:16 PDT
Created
attachment 315343
[details]
Archive of layout-test-results from ews125 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews125 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.5
Matthew Hanson
Comment 17
2017-07-13 19:04:28 PDT
Created
attachment 315394
[details]
All changes Source outside of the actual rename This patch does not include the actual rename. It will not apply cleanly until after the directories have been moved.
mitz
Comment 18
2017-07-13 19:10:02 PDT
Comment on
attachment 315394
[details]
All changes Source outside of the actual rename Read as a sequence of patches to apply on top of each other along with the directory and project rename, this looks right to me.
Matthew Hanson
Comment 19
2017-07-13 22:21:15 PDT
Resolved in:
https://trac.webkit.org/r219481
https://trac.webkit.org/r219482
https://trac.webkit.org/r219483
https://trac.webkit.org/r219484
https://trac.webkit.org/r219485
https://trac.webkit.org/r219486
https://trac.webkit.org/r219487
https://trac.webkit.org/r219488
https://trac.webkit.org/r219489
https://trac.webkit.org/r219490
https://trac.webkit.org/r219492
https://trac.webkit.org/r219493
https://trac.webkit.org/r219494
https://trac.webkit.org/r219495
https://trac.webkit.org/r219496
https://trac.webkit.org/r219497
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug