Don't call settings.GetIsGerrit() before parser.parse_args()

git_cl.py

Index: git_cl.py
===================================================================
--- git_cl.py	(revision 169567)
+++ git_cl.py	(working copy)
@@ -1246,12 +1246,16 @@
                     help="Emulate Subversion's auto properties feature.")
   parser.add_option('-c', '--use-commit-queue', action='store_true',
                     help='tell the commit queue to commit this patchset')
-  if settings.GetIsGerrit():
-    parser.add_option('--target_branch', dest='target_branch', default='master',
-                      help='target branch to upload')
+  parser.add_option('--target_branch',
+                    help='target branch to upload. ' +

[Isaac (away), 2012/11/27 08:35:09]

This help message is a bit confusing, let's clarify.  Also some spacing / style
nits.  How about:

help=('When uploading to gerrit, remote branch to '
      'use for CL.  Default: master')

[ukai, 2012/11/27 08:54:02]

Done.

+                    'Only for gerrit (default:master)')
   add_git_similarity(parser)
   (options, args) = parser.parse_args(args)
 
+  if options.target_branch and not settings.GetIsGerrit():
+    print >> sys.stderr, (
+        '\nWARNING: Use --target_branch for non gerrit repository.\n')
+
   # Print warning if the user used the -m/--message argument.  This will soon
   # change to -t/--title.
   if options.message: