Add a safeguard because I always use -p 8000 by error when I mean --port 8000.

trychange.py

 #!/usr/bin/env python
 # Copyright (c) 2011 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.
 
 """Client-side script to send a try job to the try server. It communicates to
 the try server by either writting to a svn repository or by directly connecting
 to the server by HTTP.
 """
 
@@ skipping 638 matching lines @@
   if not swallow_exception:
     if options.verbose == 0:
       logging.basicConfig(level=logging.WARNING, format=LOG_FORMAT)
     elif options.verbose == 1:
       logging.basicConfig(level=logging.INFO, format=LOG_FORMAT)
     elif options.verbose > 1:
       logging.basicConfig(level=logging.DEBUG, format=LOG_FORMAT)
 
   logging.debug(argv)
 
+  if (options.patchlevel is not None and
+      (options.patchlevel < 0 or options.patchlevel > 10)):
+    parser.error(
+        'Have you tried --port instead? You probably confused -p and -P.')

[Dirk Pranke, 2011/10/13 00:46:33]

Nit: I'd probably use something like 100 instead of 10. I've certainly seen more
than 10 patches on some issues.

[M-A Ruel, 2011/10/13 00:47:32]

No, it's patchlevel, like patch -p1, not the number of patchset.

+
   # Strip off any @ in the user, otherwise svn gets confused.
   options.user = options.user.split('@', 1)[0]
 
   if options.rietveld_url:
     # Try to extract the review number if possible and fix the protocol.
     if not '://' in options.rietveld_url:
       options.rietveld_url = 'http://' + options.rietveld_url
     match = re.match(r'^(.*)/(\d+)/?$', options.rietveld_url)
     if match:
       if options.issue or options.patchset:
@@ skipping 140 matching lines @@
     return 1
   except (gclient_utils.Error, subprocess2.CalledProcessError), e:
     print >> sys.stderr, e
     return 1
   return 0
 
 
 if __name__ == "__main__":
   fix_encoding.fix_encoding()
   sys.exit(TryChange(None, None, False))