Bug 48015

Summary: Add Git-support to do-file-rename
Product: WebKit Reporter: Daniel Bates <dbates>
Component: Tools / TestsAssignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Daniel Bates 2010-10-20 15:34:29 PDT
Currently, do-file-rename is hard coded to only work with SVN when renaming files that match the replacement criterion. Instead, we should abstract the SCM move/rename command into a common function in VCSUtils since this functionality is used by both do-file-rename and do-webcore-rename.
Comment 1 Daniel Bates 2010-10-20 15:46:03 PDT
Created attachment 71349 [details]
Patch

Add an SCM-independent move/rename function to VCSUtils.pm that can is used by both do-file-rename and do-webcore-rename.
Comment 2 Martin Robinson 2010-10-20 15:51:54 PDT
Comment on attachment 71349 [details]
Patch

Looks good!
Comment 3 Daniel Bates 2010-10-20 15:54:48 PDT
Comment on attachment 71349 [details]
Patch

Clearing flags on attachment: 71349

Committed r70187: <http://trac.webkit.org/changeset/70187>
Comment 4 Daniel Bates 2010-10-20 15:54:53 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Daniel Bates 2010-10-21 14:22:54 PDT
I inadvertently left out the line to export the function scmMoveOrRenameFile() in the patch. Committed fix in changeset 70264 <http://trac.webkit.org/changeset/70264>.