hgreview: Add support for mercurial < 1.9 (tested on 1.6)

hgreview/__init__.py

 # This file is part of hgreview.  The COPYRIGHT file at the top level of this
 # repository contains the full copyright notices and license terms.
 
 import os
 import re
 import sys
 import formatter
 import htmllib
 import urllib
 from hashlib import md5
 
-from mercurial import scmutil, patch, mdiff, copies, node, commands
+from mercurial.__version__ import version as mercurial_version
+from mercurial import patch, mdiff, copies, node, commands
+
+try:
+    test = map(int, mercurial_version.split('.')) >= [1, 9]
+except ValueError:
+    test = True
+if test:
+    from mercurial.scmutil import revpair, matchfiles
+else:
+    from mercurial.cmdutil import revpair, matchfiles
+
 
 from rietveld import (GetEmail, GetRpcServer, CheckReviewer, MAX_UPLOAD_SIZE,
     EncodeMultipartFormData, UploadSeparatePatches, UploadBaseFiles)
 
 
 RAW_PATCH_HREF = re.compile('.*/issue[0-9]+_([0-9])+.diff$')
 
 
 class CodereviewParser(htmllib.HTMLParser):
 
@@ skipping 25 matching lines @@
     issue_file = _get_issue_file(repo)
     if os.path.isfile(issue_file):
         return open(issue_file, 'r').read().strip()
 
 def _get_server(ui):
     return ui.config('review', 'server',
         default='http://codereview.appspot.com')
 
 def review(ui, repo, *args, **opts):
     revs = [opts['rev']] if opts['rev'] else []
-    node1, node2 = scmutil.revpair(repo, revs)
+    node1, node2 = revpair(repo, revs)
     modified, added, removed, deleted, unknown, ignored, clean = \
             repo.status(node1, node2, unknown=True)
     if opts['fetch']:
         if modified or added or removed or deleted or unknown:
             ui.warn('The repository is not clean.', '\n')
             sys.exit(1)
         if not opts.get('issue'):
             issue_id = _get_issue_id(repo)
             if not issue_id:
                 ui.status('No .hg/review_id found', '\n')
@@ skipping 63 matching lines @@
 
     # getting informations about copied/moved files
     copymove_info = copies.copies(repo, base_rev, current_rev, null_rev)[0]
     for newname, oldname in copymove_info.items():
         oldcontent = base_rev[oldname].data()
         newcontent = current_rev[newname].data()
         is_binary = "\0" in oldcontent or "\0" in newcontent
         files[newname] = (oldcontent, newcontent, is_binary, 'M')
 
     # modified files
-    for filename in scmutil.matchfiles(repo, modified):
+    for filename in matchfiles(repo, modified):
         oldcontent = base_rev[filename].data()
         newcontent = current_rev[filename].data()
         is_binary = "\0" in oldcontent or "\0" in newcontent
         files[filename] = (oldcontent, newcontent, is_binary, 'M')
 
     # added files
-    for filename in scmutil.matchfiles(repo, added):
+    for filename in matchfiles(repo, added):
         oldcontent = ''
         newcontent = current_rev[filename].data()
         is_binary = "\0" in newcontent
         files[filename] = (oldcontent, newcontent, is_binary, 'A')
 
     # removed files
-    for filename in scmutil.matchfiles(repo, removed):
+    for filename in matchfiles(repo, removed):
         if filename in copymove_info.values():
             # file has been moved or copied
             continue
         oldcontent = base_rev[filename].data()
         newcontent = ''
         is_binary = "\0" in oldcontent
         files[filename] = (oldcontent, newcontent, is_binary, 'R')
 
     server = _get_server(ui)
     ui.status('Server used %s' % server, '\n')
@@ skipping 81 matching lines @@
         ('r', 'reviewers', [], 'Add reviewers'),
         ('i', 'issue', '', 'Issue number. Defaults to new issue'),
         ('m', 'message', '', 'Codereview message'),
         ('', 'rev', '', 'Revision number to diff against'),
         ('', 'send_email', None, 'Send notification email to reviewers'),
         ('', 'id', None, 'ouput issue id'),
         ('', 'url', None, 'ouput issue URL'),
         ('', 'fetch', None, 'Fetch patch and apply to repository'),
     ], "hg review [options]"),
 }