Bug 72349 – VCSUtils.pm doesn't handle svn 1.7 correctly

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Peter Kasting]

$isSVNDirectory fails on svn 1.7 in non-root directories because svn 1.7 eliminates the .svn subdirectories everywhere but the root.

I'm going to fix this by blindly copying the git code, which instead shells out to git to do the work, and adapt it to use "svn info".

[Peter Kasting]

Created attachment 115089 [details]
patch v1

This seems to work on my machine (tested both positively and negatively).  Did not test on svn 1.6 or below.

[Adam Roben (:aroben)]

Comment on attachment 115089 [details]
patch v1

View in context: https://bugs.webkit.org/attachment.cgi?id=115089&action=review

> Tools/Scripts/VCSUtils.pm:252
> +    return system("cd $dir && svn info > " . File::Spec->devnull() . " 2>&1") == 0;

I wonder if this will cause problems for folks using native Win32 Perl?

[Adam Roben (:aroben)]

Simon, any idea if this will be a problem for Qt/Win?

[Simon Hausmann]

(In reply to comment #3)
> Simon, any idea if this will be a problem for Qt/Win?

Don't think so :)

[WebKit Review Bot]

Comment on attachment 115089 [details]
patch v1

Clearing flags on attachment: 115089

Committed r100286: <http://trac.webkit.org/changeset/100286>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.