Summary: | Rename WebKitTools to Tools | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||
Component: | Tools / Tests | Assignee: | mitz | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | dpranke, eric, jeffm, maruel, nsylvain | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
mitz
2010-11-20 00:13:31 PST
Created attachment 74473 [details]
Rename WebKitTools to Tools and update references
Comment on attachment 74473 [details]
Rename WebKitTools to Tools and update references
rs=me
This change looks mostly fine and I think it's a great idea, but it's likely to break stuff outside of the WebKit repo. It will certainly break some wrapper scripts in Chromium that we use to minimize the dependencies of our buildbots on WebKit directory layout. Although it looks like SVN doesn't really support symlinks on windows, did you consider the idea of adding a symlink from WebKitTools -> Tools to at least extend the transition period for other repos on non-Win platforms? OpenSource/Tools/vcbin/midl.exe needs to be rebuilt as well, since it's checked in and has a hard-coded reference to the old name of this folder (WebKitTools). Looks like the source code in MIDLWrapper.cpp was updated, but not the built tool. I filed https://bugs.webkit.org/show_bug.cgi?id=51347 to rebuild midl.exe. |