diff options
author | Patrick M?zard <patrick@mezard.eu> | 2014-02-24 10:11:37 -0500 |
---|---|---|
committer | Patrick M?zard <patrick@mezard.eu> | 2014-02-24 10:11:37 -0500 |
commit | 1d48ac641e9b85bfff3b9c185a7ff0f2af573419 (patch) | |
tree | e9f179166cf4cd78b5b0e239c6782b14578d2cec /lib/codereview | |
parent | 1d3337d6117e778b00d6d875575fd5f5f5a0b011 (diff) | |
download | go-1d48ac641e9b85bfff3b9c185a7ff0f2af573419.tar.gz |
codereview: remove unused upload_options.revision
LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://codereview.appspot.com/60640052
Committer: Russ Cox <rsc@golang.org>
Diffstat (limited to 'lib/codereview')
-rw-r--r-- | lib/codereview/codereview.py | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/lib/codereview/codereview.py b/lib/codereview/codereview.py index e19d601ff..67341ebc3 100644 --- a/lib/codereview/codereview.py +++ b/lib/codereview/codereview.py @@ -2656,7 +2656,6 @@ def RietveldSetup(ui, repo): upload_options.message = None upload_options.issue = None upload_options.download_base = False - upload_options.revision = None upload_options.send_mail = False upload_options.vcs = None upload_options.server = server @@ -3435,18 +3434,16 @@ class MercurialVCS(VersionControlSystem): cwd = os.path.normpath(os.getcwd()) assert cwd.startswith(self.repo_dir) self.subdir = cwd[len(self.repo_dir):].lstrip(r"\/") - if self.options.revision: - self.base_rev = self.options.revision + mqparent, err = RunShellWithReturnCode(['hg', 'log', '--rev', 'qparent', '--template={node}']) + if not err and mqparent != "": + self.base_rev = mqparent else: - mqparent, err = RunShellWithReturnCode(['hg', 'log', '--rev', 'qparent', '--template={node}']) - if not err and mqparent != "": - self.base_rev = mqparent - else: - out = RunShell(["hg", "parents", "-q"], silent_ok=True).strip() - if not out: - # No revisions; use 0 to mean a repository with nothing. - out = "0:0" - self.base_rev = out.split(':')[1].strip() + out = RunShell(["hg", "parents", "-q"], silent_ok=True).strip() + if not out: + # No revisions; use 0 to mean a repository with nothing. + out = "0:0" + self.base_rev = out.split(':')[1].strip() + def _GetRelPath(self, filename): """Get relative path of a file according to the current directory, given its logical path in the repo.""" |