Bug 28881 – svn-create-patch should check if the repo path is the same when trying to find the root

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Anders Carlsson]

Currently, when svn-create-patch checks for the repository root it only checks if the repository UUID is the same. It needs to check that the repository path is the same as well.

[David Kilzer (:ddkilzer)]

svn-create-patch switched from repository root to UUID in r46134.

<http://trac.webkit.org/changeset/46134>

For situations where you have two nested working directories that are from the same repository (but at different repository roots), it's necessary to check both the repository root and the UUID.

[David Kilzer (:ddkilzer)]

Created attachment 38881 [details]
Patch v1

[Eric Seidel (no email)]

Comment on attachment 38881 [details]
Patch v1

Looks good to me.  So sad that we don't have unit tests for this sort of thing. :(  It's possible to test this stuff via scm.py for sure.  cq- for ChangeLog patch and because yor'e a committer.

[Adam Barth]

We have this bug in one of the bugzilla-tool python modules too (probably scm.py).

[David Kilzer (:ddkilzer)]

Comment on attachment 38881 [details]
Patch v1

(In reply to comment #4)
> We have this bug in one of the bugzilla-tool python modules too (probably
> scm.py).

Will fix scm.py as well.

[David Kilzer (:ddkilzer)]

(In reply to comment #3)
> (From update of attachment 38881 [details])
> Looks good to me.  So sad that we don't have unit tests for this sort of thing.
> :(  It's possible to test this stuff via scm.py for sure.  cq- for ChangeLog
> patch and because yor'e a committer.

(In reply to comment #4)
> We have this bug in one of the bugzilla-tool python modules too (probably
> scm.py).

I filed Bug 28925 instead.  Don't have time to work on this right now, and it's not causing any known issues for WebKit contributors.

[David Kilzer (:ddkilzer)]

Committed r48003: <http://trac.webkit.org/changeset/48003>