Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(883)

Unified Diff: git_footers.py

Issue 2354313003: Remove svn support from smaller git extensions (Closed)
Patch Set: rebase Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « git_drover.py ('k') | tests/git_footers_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: git_footers.py
diff --git a/git_footers.py b/git_footers.py
index 412e61fd25526767e382521343a3d70a2b4aed26..3df4f82a6ad0594c3b7551b87a3ff319e41ca913 100755
--- a/git_footers.py
+++ b/git_footers.py
@@ -15,7 +15,6 @@ import git_common as git
FOOTER_PATTERN = re.compile(r'^\s*([\w-]+): (.*)$')
CHROME_COMMIT_POSITION_PATTERN = re.compile(r'^([\w/\-\.]+)@{#(\d+)}$')
-GIT_SVN_ID_PATTERN = re.compile('^([^\s@]+)@(\d+)')
def normalize_name(header):
@@ -68,20 +67,6 @@ def split_footers(message):
return message_lines[:-len(footer_lines)], footer_lines, footers
-def get_footer_svn_id(branch=None):
- if not branch:
- branch = git.root()
- svn_id = None
- message = git.run('log', '-1', '--format=%B', branch)
- footers = parse_footers(message)
- git_svn_id = get_unique(footers, 'git-svn-id')
- if git_svn_id:
- match = GIT_SVN_ID_PATTERN.match(git_svn_id)
- if match:
- svn_id = match.group(1)
- return svn_id
-
-
def get_footer_change_id(message):
"""Returns a list of Gerrit's ChangeId from given commit message."""
return parse_footers(message).get(normalize_name('Change-Id'), [])
@@ -150,9 +135,7 @@ def get_position(footers):
Cr-Commit-Position: refs/heads/master@{#292272}
- would give the return value ('refs/heads/master', 292272). If
- Cr-Commit-Position is not defined, we try to infer the ref and position
- from git-svn-id. The position number can be None if it was not inferrable.
+ would give the return value ('refs/heads/master', 292272).
"""
position = get_unique(footers, 'Cr-Commit-Position')
@@ -161,29 +144,6 @@ def get_position(footers):
assert match, 'Invalid Cr-Commit-Position value: %s' % position
return (match.group(1), match.group(2))
- svn_commit = get_unique(footers, 'git-svn-id')
- if svn_commit:
- match = GIT_SVN_ID_PATTERN.match(svn_commit)
- assert match, 'Invalid git-svn-id value: %s' % svn_commit
- # V8 has different semantics than Chromium.
- if re.match(r'.*https?://v8\.googlecode\.com/svn/trunk',
- match.group(1)):
- return ('refs/heads/candidates', match.group(2))
- if re.match(r'.*https?://v8\.googlecode\.com/svn/branches/bleeding_edge',
- match.group(1)):
- return ('refs/heads/master', match.group(2))
-
- # Assume that any trunk svn revision will match the commit-position
- # semantics.
- if re.match('.*/trunk.*$', match.group(1)):
- return ('refs/heads/master', match.group(2))
-
- # But for now only support faking branch-heads for chrome.
- branch_match = re.match('.*/chrome/branches/([\w/-]+)/src$', match.group(1))
- if branch_match:
- # svn commit numbers do not map to branches.
- return ('refs/branch-heads/%s' % branch_match.group(1), None)
-
raise ValueError('Unable to infer commit position from footers')
« no previous file with comments | « git_drover.py ('k') | tests/git_footers_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698