Fixes 4662 - [hgreview] Add an option for suppressing the email after upload

hgreview.py

 import BaseHTTPServer
 import os
 import re
 import socket
 import sys
 import urllib
 
 from mercurial import cmdutil, error
 
 SERVER = 'https://codereview.adblockplus.org'
 UPLOADTOOL_URL = SERVER + '/static/upload.py'
 
 cmdtable = {}
 command = cmdutil.command(cmdtable)
 
 
 @command('review',
          [
              ('i', 'issue', '', 'If given, adds a patch set to this review, otherwise create a new one.', 'ISSUE'),
              ('r', 'revision', '', 'Revision to diff against or a revision range to upload.', 'REV'),
              ('c', 'change', '', 'A single revision to upload.', 'REV'),
              ('t', 'title', '', 'New review subject or new patch set title.', 'TITLE'),
              ('m', 'message', '', 'New review description or new patch set message.', 'MESSAGE'),
              ('w', 'reviewers', '', 'Add reviewers (comma separated email addresses or @adblockplus.org user names).', 'REVIEWERS'),
              ('', 'cc', '', 'Add CC (comma separated email addresses or @adblockplus.org user names).', 'CC'),
              ('', 'private', None, 'Make the review restricted to reviewers and those CCed.'),
              ('y', 'assume_yes', None, 'Assume that the answer to yes/no questions is \'yes\'.'),
              ('', 'print_diffs', None, 'Print full diffs.'),
+             ('', 'no_mail', None, 'Don\'t send an email after uploading.'),
          ], '[options] [path...]')
 def review(ui, repo, *paths, **opts):
     '''
       Uploads a review to https://codereview.adblockplus.org/ or updates an
       existing review request. This will always send mails for new reviews, when
       updating a review mails will only be sent if a message is given.
     '''
     args = ['--oauth2', '--server', SERVER]
     if ui.debugflag:
         args.append('--noisy')
     elif ui.verbose:
         args.append('--verbose')
     elif ui.quiet:
         args.append('--quiet')
 
-    if not opts.get('issue') or opts.get('message'):
+    if (not opts.get('no_mail') and
+        (not opts.get('issue') or opts.get('message'))):
         args.append('--send_mail')
 
     if opts.get('revision') and opts.get('change'):
         raise error.Abort('Ambiguous revision range, only one of --revision and --change can be specified.')
     if opts.get('change'):
         rev = repo[opts['change']]
         args.extend(['--rev', '{}:{}'.format(rev.parents()[0], rev)])
     elif opts.get('revision'):
         args.extend(['--rev', opts['revision']])
     else:
@@ skipping 13 matching lines @@
 
         path = (ui.config('paths', 'default-push')
                 or ui.config('paths', 'default')
                 or '')
         match = re.search(r'^(?:https://|ssh://hg@)(.*)', path)
         if match:
             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.')
+            if opts.get('no_mail'):
+                print 'No reviewers specified, edit the review to add some.'

[Wladimir Palant, 2016/11/23 16:30:14]

Please use ui.status() rather than print, this will make sure that the output is
suppressed if -q is specified.

[Felix Dahlke, 2016/11/23 17:21:26]

Done.

+            else:
+                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', 'base_url'):
         if opts.get(opt, ''):
             args.extend(['--' + opt, opts[opt]])
@@ skipping 58 matching lines @@
   </body>
 </html>
 ''' % port
 
     # Run the upload tool
     issue, patchset = scope['RealMain']([upload_path] + args)
 
     # Wait for the page to check in and retrieve issue URL
     if server:
         server.handle_request()