Fix using migrate-downstream-fork with up-to-date monorepo #19
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Commit 2f2aeb9b has the following in its log:
llvm-svn: 181277
llvm-svn: 181274
llvm-svn: 181273
I assume this is because it is a commit that exists in 3 subprojects, and
thus has 3 separate svn revisions which were combined during monorepo
conversion.
find_svnrev assumes a 1-to-1 git commit to svn revision correspondance
which seems inaccurate with how the monorepo is now laid out.
This commit now has the function return a list of tuples, so that we can
represent the 1-to-many git commit to svn revision relationship.
This also required a change to the regex, which used '\Z' to match the
assumed 1-to-1 svn revision associated with the commit. This assumption is
no longer true in the monorepo. The new strategy is to start at the end of
the message and work our way up, matching each line as an svn revision
until we reach the first non-match.
Fixes #18