commit b0fbc7fb58af8ee1875993daad481b87bf8c2f94
author Mike Frysinger <vapier@google.com> Tue Feb 25 02:58:04 2020 -0500
committer Mike Frysinger <vapier@google.com> Tue Feb 25 20:58:09 2020 +0000
tree 8c433776a7fed38dffbe208c8a8d4d976f626bae
parent 4c418bf423300b0cc4c82ce9ecca5579274a29fa

upload: drop support for drafts

Draft CLs were replaced by private/wip CLs in Gerrit instead years ago.

Change-Id: If4f3d6606aad40a6f1617a49681dfd45c64d3d37
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/256673
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Tested-by: Mike Frysinger <vapier@google.com>

project.py

diff --git a/project.py b/project.py
index 66a4b3b..b93dcd5 100644
--- a/project.py
+++ b/project.py
@@ -202,7 +202,6 @@
                       auto_topic=False,
                       hashtags=(),
                       labels=(),
-                      draft=False,
                       private=False,
                       notify=None,
                       wip=False,
@@ -215,7 +214,6 @@
                                  auto_topic=auto_topic,
                                  hashtags=hashtags,
                                  labels=labels,
-                                 draft=draft,
                                  private=private,
                                  notify=notify,
                                  wip=wip,
@@ -1349,7 +1347,6 @@
                       auto_topic=False,
                       hashtags=(),
                       labels=(),
-                      draft=False,
                       private=False,
                       notify=None,
                       wip=False,
@@ -1399,12 +1396,7 @@
     if dest_branch.startswith(R_HEADS):
       dest_branch = dest_branch[len(R_HEADS):]
 
-    upload_type = 'for'
-    if draft:
-      upload_type = 'drafts'
-
-    ref_spec = '%s:refs/%s/%s' % (R_HEADS + branch.name, upload_type,
-                                  dest_branch)
+    ref_spec = '%s:refs/for/%s' % (R_HEADS + branch.name, dest_branch)
     opts = []
     if auto_topic:
       opts += ['topic=' + branch.name]

subcmds/upload.py

diff --git a/subcmds/upload.py b/subcmds/upload.py
index 93f9c1e..9c4337d 100644
--- a/subcmds/upload.py
+++ b/subcmds/upload.py
@@ -173,9 +173,6 @@
     p.add_option('--cbr', '--current-branch',
                  dest='current_branch', action='store_true',
                  help='Upload current git branch.')
-    p.add_option('-d', '--draft',
-                 action='store_true', dest='draft', default=False,
-                 help='If specified, upload as a draft.')
     p.add_option('--ne', '--no-emails',
                  action='store_false', dest='notify', default=True,
                  help='If specified, do not send emails on upload.')
@@ -247,7 +244,7 @@
 
       destination = opt.dest_branch or project.dest_branch or project.revisionExpr
       print('Upload project %s/ to remote branch %s%s:' %
-            (project.relpath, destination, ' (draft)' if opt.draft else ''))
+            (project.relpath, destination, ' (private)' if opt.private else ''))
       print('  branch %s (%2d commit%s, %s):' % (
           name,
           len(commit_list),
@@ -468,7 +465,6 @@
                                auto_topic=opt.auto_topic,
                                hashtags=hashtags,
                                labels=labels,
-                               draft=opt.draft,
                                private=opt.private,
                                notify=None if opt.notify else 'NONE',
                                wip=opt.wip,