diff options
Diffstat (limited to 'subcmds/upload.py')
| -rw-r--r-- | subcmds/upload.py | 41 |
1 files changed, 23 insertions, 18 deletions
diff --git a/subcmds/upload.py b/subcmds/upload.py index 925652c2..a6ada337 100644 --- a/subcmds/upload.py +++ b/subcmds/upload.py | |||
| @@ -13,6 +13,7 @@ | |||
| 13 | # See the License for the specific language governing permissions and | 13 | # See the License for the specific language governing permissions and |
| 14 | # limitations under the License. | 14 | # limitations under the License. |
| 15 | 15 | ||
| 16 | from __future__ import print_function | ||
| 16 | import copy | 17 | import copy |
| 17 | import re | 18 | import re |
| 18 | import sys | 19 | import sys |
| @@ -26,16 +27,18 @@ UNUSUAL_COMMIT_THRESHOLD = 5 | |||
| 26 | 27 | ||
| 27 | def _ConfirmManyUploads(multiple_branches=False): | 28 | def _ConfirmManyUploads(multiple_branches=False): |
| 28 | if multiple_branches: | 29 | if multiple_branches: |
| 29 | print "ATTENTION: One or more branches has an unusually high number of commits." | 30 | print('ATTENTION: One or more branches has an unusually high number' |
| 31 | 'of commits.') | ||
| 30 | else: | 32 | else: |
| 31 | print "ATTENTION: You are uploading an unusually high number of commits." | 33 | print('ATTENTION: You are uploading an unusually high number of commits.') |
| 32 | print "YOU PROBABLY DO NOT MEAN TO DO THIS. (Did you rebase across branches?)" | 34 | print('YOU PROBABLY DO NOT MEAN TO DO THIS. (Did you rebase across' |
| 35 | 'branches?)') | ||
| 33 | answer = raw_input("If you are sure you intend to do this, type 'yes': ").strip() | 36 | answer = raw_input("If you are sure you intend to do this, type 'yes': ").strip() |
| 34 | return answer == "yes" | 37 | return answer == "yes" |
| 35 | 38 | ||
| 36 | def _die(fmt, *args): | 39 | def _die(fmt, *args): |
| 37 | msg = fmt % args | 40 | msg = fmt % args |
| 38 | print >>sys.stderr, 'error: %s' % msg | 41 | print('error: %s' % msg, file=sys.stderr) |
| 39 | sys.exit(1) | 42 | sys.exit(1) |
| 40 | 43 | ||
| 41 | def _SplitEmails(values): | 44 | def _SplitEmails(values): |
| @@ -176,14 +179,14 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 176 | date = branch.date | 179 | date = branch.date |
| 177 | commit_list = branch.commits | 180 | commit_list = branch.commits |
| 178 | 181 | ||
| 179 | print 'Upload project %s/ to remote branch %s:' % (project.relpath, project.revisionExpr) | 182 | print('Upload project %s/ to remote branch %s:' % (project.relpath, project.revisionExpr)) |
| 180 | print ' branch %s (%2d commit%s, %s):' % ( | 183 | print(' branch %s (%2d commit%s, %s):' % ( |
| 181 | name, | 184 | name, |
| 182 | len(commit_list), | 185 | len(commit_list), |
| 183 | len(commit_list) != 1 and 's' or '', | 186 | len(commit_list) != 1 and 's' or '', |
| 184 | date) | 187 | date)) |
| 185 | for commit in commit_list: | 188 | for commit in commit_list: |
| 186 | print ' %s' % commit | 189 | print(' %s' % commit) |
| 187 | 190 | ||
| 188 | sys.stdout.write('to %s (y/N)? ' % remote.review) | 191 | sys.stdout.write('to %s (y/N)? ' % remote.review) |
| 189 | answer = sys.stdin.readline().strip().lower() | 192 | answer = sys.stdin.readline().strip().lower() |
| @@ -317,7 +320,7 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 317 | sys.stdout.write('Uncommitted changes in ' + branch.project.name + ' (did you forget to amend?). Continue uploading? (y/N) ') | 320 | sys.stdout.write('Uncommitted changes in ' + branch.project.name + ' (did you forget to amend?). Continue uploading? (y/N) ') |
| 318 | a = sys.stdin.readline().strip().lower() | 321 | a = sys.stdin.readline().strip().lower() |
| 319 | if a not in ('y', 'yes', 't', 'true', 'on'): | 322 | if a not in ('y', 'yes', 't', 'true', 'on'): |
| 320 | print >>sys.stderr, "skipping upload" | 323 | print("skipping upload", file=sys.stderr) |
| 321 | branch.uploaded = False | 324 | branch.uploaded = False |
| 322 | branch.error = 'User aborted' | 325 | branch.error = 'User aborted' |
| 323 | continue | 326 | continue |
| @@ -334,8 +337,8 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 334 | branch.uploaded = False | 337 | branch.uploaded = False |
| 335 | have_errors = True | 338 | have_errors = True |
| 336 | 339 | ||
| 337 | print >>sys.stderr, '' | 340 | print(file=sys.stderr) |
| 338 | print >>sys.stderr, '----------------------------------------------------------------------' | 341 | print('----------------------------------------------------------------------', file=sys.stderr) |
| 339 | 342 | ||
| 340 | if have_errors: | 343 | if have_errors: |
| 341 | for branch in todo: | 344 | for branch in todo: |
| @@ -344,17 +347,19 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 344 | fmt = ' (%s)' | 347 | fmt = ' (%s)' |
| 345 | else: | 348 | else: |
| 346 | fmt = '\n (%s)' | 349 | fmt = '\n (%s)' |
| 347 | print >>sys.stderr, ('[FAILED] %-15s %-15s' + fmt) % ( | 350 | print(('[FAILED] %-15s %-15s' + fmt) % ( |
| 348 | branch.project.relpath + '/', \ | 351 | branch.project.relpath + '/', \ |
| 349 | branch.name, \ | 352 | branch.name, \ |
| 350 | str(branch.error)) | 353 | str(branch.error)), |
| 351 | print >>sys.stderr, '' | 354 | file=sys.stderr) |
| 355 | print() | ||
| 352 | 356 | ||
| 353 | for branch in todo: | 357 | for branch in todo: |
| 354 | if branch.uploaded: | 358 | if branch.uploaded: |
| 355 | print >>sys.stderr, '[OK ] %-15s %s' % ( | 359 | print('[OK ] %-15s %s' % ( |
| 356 | branch.project.relpath + '/', | 360 | branch.project.relpath + '/', |
| 357 | branch.name) | 361 | branch.name), |
| 362 | file=sys.stderr) | ||
| 358 | 363 | ||
| 359 | if have_errors: | 364 | if have_errors: |
| 360 | sys.exit(1) | 365 | sys.exit(1) |
| @@ -385,7 +390,7 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 385 | try: | 390 | try: |
| 386 | hook.Run(opt.allow_all_hooks, project_list=pending_proj_names) | 391 | hook.Run(opt.allow_all_hooks, project_list=pending_proj_names) |
| 387 | except HookError as e: | 392 | except HookError as e: |
| 388 | print >>sys.stderr, "ERROR: %s" % str(e) | 393 | print("ERROR: %s" % str(e), file=sys.stderr) |
| 389 | return | 394 | return |
| 390 | 395 | ||
| 391 | if opt.reviewers: | 396 | if opt.reviewers: |
| @@ -395,7 +400,7 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 395 | people = (reviewers,cc) | 400 | people = (reviewers,cc) |
| 396 | 401 | ||
| 397 | if not pending: | 402 | if not pending: |
| 398 | print >>sys.stdout, "no branches ready for upload" | 403 | print("no branches ready for upload", file=sys.stderr) |
| 399 | elif len(pending) == 1 and len(pending[0][1]) == 1: | 404 | elif len(pending) == 1 and len(pending[0][1]) == 1: |
| 400 | self._SingleBranch(opt, pending[0][1][0], people) | 405 | self._SingleBranch(opt, pending[0][1][0], people) |
| 401 | else: | 406 | else: |
