Gerrit git cl upload: obey -s --send-mail option.

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld and Gerrit."""
 
 from distutils.version import LooseVersion
@@ skipping 2450 matching lines @@
     # Extra options that can be specified at push time. Doc:
     # https://gerrit-review.googlesource.com/Documentation/user-upload.html
     refspec_opts = []
     if options.title:
       # Per doc, spaces must be converted to underscores, and Gerrit will do the
       # reverse on its side.
       if '_' in options.title:
         print('WARNING: underscores in title will be converted to spaces.')
       refspec_opts.append('m=' + options.title.replace(' ', '_'))
 
+    if options.send_mail:
+      if not change_desc.get_reviewers():
+        DieWithError('Must specify reviewers to send email.')
+      refspec_opts.append('notify=ALL')
+    else:
+      refspec_opts.append('notify=NONE')
+
     cc = self.GetCCList().split(',')
     if options.cc:
       cc.extend(options.cc)
     cc = filter(None, cc)
     if cc:
       # refspec_opts.extend('cc=' + email.strip() for email in cc)
       # TODO(tandrii): enable this back. http://crbug.com/604377
       print('WARNING: Gerrit doesn\'t yet support cc-ing arbitrary emails.\n'
             '         Ignoring cc-ed emails. See http://crbug.com/604377.')
 
     if change_desc.get_reviewers():
       refspec_opts.extend('r=' + email.strip()
                           for email in change_desc.get_reviewers())
 
-
     refspec_suffix = ''
     if refspec_opts:
       refspec_suffix = '%' + ','.join(refspec_opts)
       assert ' ' not in refspec_suffix, (
           'spaces not allowed in refspec: "%s"' % refspec_suffix)
     refspec = '%s:refs/for/%s%s' % (ref_to_push, branch, refspec_suffix)
 
     push_stdout = gclient_utils.CheckCallAndFilter(
         ['git', 'push', gerrit_remote, refspec],
         print_stdout=True,
@@ skipping 2405 matching lines @@
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   setup_color.init()
   try:
     sys.exit(main(sys.argv[1:]))
   except KeyboardInterrupt:
     sys.stderr.write('interrupted\n')
     sys.exit(1)