Issue 4388 - [hgreview] Reviews should use Base URL field in order to indicate the repository

hgreview.py

Index: hgreview.py
===================================================================
--- a/hgreview.py
+++ b/hgreview.py
@@ -65,34 +65,31 @@ def review(ui, repo, *paths, **opts):
         if not opts.get('message'):
             opts['message'] = opts['title']
 
         path = (ui.config('paths', 'default-push')
                 or ui.config('paths', 'default')
                 or '')
         match = re.search(r'^(?:https://|ssh://hg@)(.*)', path)
         if match:
-            opts['message'] = '{0}\n\nRepository: {1}'.format(
-                opts['message'].strip(),
-                match.group(1)
-            )
+            opts['base_url'] = 'https://' + match.group(1)
 
         # Make sure there is at least one reviewer
         if not opts.get('reviewers'):
             opts['reviewers'] = ui.prompt('Reviewers (comma-separated): ', '')
         if not opts['reviewers'].strip():
             raise error.Abort('No reviewers given.')
 
     for opt in ('reviewers', 'cc'):
         if opts.get(opt):
             users = [u if '@' in u else u + '@adblockplus.org'
                      for u in re.split(r'\s*,\s*', opts[opt])]
             opts[opt] = ','.join(users)
 
-    for opt in ('issue', 'title', 'message', 'reviewers', 'cc'):
+    for opt in ('issue', 'title', 'message', 'reviewers', 'cc', 'base_url'):
         if opts.get(opt, ''):
             args.extend(['--' + opt, opts[opt]])
 
     for opt in ('private', 'assume_yes', 'print_diffs'):
         if opts.get(opt, False):
             args.append('--' + opt)
 
     args.extend(paths)