Bug 27328 – WebKitTools/Scripts/svn-create-patch is broken

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Jeremy Orlow]

https://bugs.webkit.org/show_bug.cgi?id=26999 broke svn-create-patch for me

jorlow@jorlow-laptop:/Files/src/webkit
$ WebKitTools/Scripts/svn-create-patch 
svn: '/WebCore/ChangeLog' is not a working copy
svn: '/WebCore/ChangeLog' does not exist
Use of uninitialized value in pattern match (m//) at WebKitTools/Scripts/svn-create-patch line 213.

I rolled it back to the previous version and it worked.  Let me know there's any more information I can provide.

[Joseph Pecoraro]

It seems like its giving you an Absolute Path from Root:
  "/WebCore/ChangeLog"

Instead of just:
  "WebCore/ChangeLog"

Let me look into why that might have happened (because unfortunately it doesn't do that on my machine).

[Joseph Pecoraro]

Hmm.  Well the svn error comes from `svn diff '/path/to/file'` when the file doesn't exist.  That error entirely makes sense because "/WebCore/ChangeLog" doesn't exist.  However the leading slash makes no sense to me and I can't reproduce it on my machine (Mac OS X Perl 5.8.8).

On my system:

  shell> perl -e 'use File::Spec; print File::Spec->abs2rel("/blah","/blah"), "\n"'
  .

Try running that on your system to see if you get something other then ".".  If that is the case then I can stop scratching my head.

Or we could just always strip a leading slash from $prefix because it should never happen anyways.  What do the stronger perl programmers think?

[Jeremy Orlow]

"""
jorlow@jorlow-laptop:/Files/src/webkit
$ perl -e 'use File::Spec; print File::Spec->abs2rel("/blah","/blah"), "\n"'
.
"""

In other words, it printed ".".

Note that the script runs when I do it inside WebCore...just not in the root directory of the tree.

[Joseph Pecoraro]

Created attachment 32838 [details]
Fix for Unusual Perl Behavior

Perl seems to have been acting differently on our machines.  This handles the special case that was acting differently.  Its been tested on Jeremy Orlow's system (thanks for taking the time!) and worked.

[Adam Barth]

Will land.

[Adam Barth]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/svn-create-patch
Committed r45985
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/svn-create-patch
r45985 = 4bf341915a24384819820406067959d02c3eeab8 (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/45985