 Chromium Code Reviews
 Chromium Code Reviews| OLD | NEW | 
|---|---|
| 1 #!/usr/bin/python | 1 #!/usr/bin/python | 
| 2 # Copyright (c) 2009 The Chromium Authors. All rights reserved. | 2 # Copyright (c) 2009 The Chromium Authors. All rights reserved. | 
| 3 # Use of this source code is governed by a BSD-style license that can be | 3 # Use of this source code is governed by a BSD-style license that can be | 
| 4 # found in the LICENSE file. | 4 # found in the LICENSE file. | 
| 5 """Client-side script to send a try job to the try server. It communicates to | 5 """Client-side script to send a try job to the try server. It communicates to | 
| 6 the try server by either writting to a svn repository or by directly connecting | 6 the try server by either writting to a svn repository or by directly connecting | 
| 7 to the server by HTTP. | 7 to the server by HTTP. | 
| 8 """ | 8 """ | 
| 9 | 9 | 
| 10 | 10 | 
| (...skipping 434 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 445 default_settings.get('default_transport')) | 445 default_settings.get('default_transport')) | 
| 446 | 446 | 
| 447 # Parse argv | 447 # Parse argv | 
| 448 parser = optparse.OptionParser(usage=USAGE, | 448 parser = optparse.OptionParser(usage=USAGE, | 
| 449 version=__version__, | 449 version=__version__, | 
| 450 prog=prog) | 450 prog=prog) | 
| 451 | 451 | 
| 452 group = optparse.OptionGroup(parser, "Result and status") | 452 group = optparse.OptionGroup(parser, "Result and status") | 
| 453 group.add_option("-u", "--user", default=getpass.getuser(), | 453 group.add_option("-u", "--user", default=getpass.getuser(), | 
| 454 help="Owner user name [default: %default]") | 454 help="Owner user name [default: %default]") | 
| 455 group.add_option("-e", "--email", default=os.environ.get('EMAIL_ADDRESS'), | 455 group.add_option("-e", "--email", | 
| 456 help="Email address where to send the results. Use the " | 456 default=os.environ.get('TRYBOT_RESULTS_EMAIL_ADDRESS', | 
| 457 "EMAIL_ADDRESS environment variable to set the default " | 457 os.environ.get('EMAIL_ADDRESS')), | 
| 
M-A Ruel
2009/11/09 16:40:09
actually, this line should be +4 but not a big dea
 | |
| 458 "email address [default: %default]") | 458 help="Email address where to send the results. Use either " | 
| 459 "the TRYBOT_RESULTS_EMAIL_ADDRESS environment " | |
| 460 "variable or EMAIL_ADDRESS to set the email address " | |
| 461 "the try bots report results to [default: %default]") | |
| 459 group.add_option("-n", "--name", | 462 group.add_option("-n", "--name", | 
| 460 help="Descriptive name of the try job") | 463 help="Descriptive name of the try job") | 
| 461 group.add_option("--issue", type='int', | 464 group.add_option("--issue", type='int', | 
| 462 help="Update rietveld issue try job status") | 465 help="Update rietveld issue try job status") | 
| 463 group.add_option("--patchset", type='int', | 466 group.add_option("--patchset", type='int', | 
| 464 help="Update rietveld issue try job status") | 467 help="Update rietveld issue try job status") | 
| 465 parser.add_option_group(group) | 468 parser.add_option_group(group) | 
| 466 | 469 | 
| 467 group = optparse.OptionGroup(parser, "Try job options") | 470 group = optparse.OptionGroup(parser, "Try job options") | 
| 468 group.add_option("-b", "--bot", action="append", | 471 group.add_option("-b", "--bot", action="append", | 
| (...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 583 False, | 586 False, | 
| 584 sys.stdout) | 587 sys.stdout) | 
| 585 | 588 | 
| 586 if options.name is None: | 589 if options.name is None: | 
| 587 if options.issue: | 590 if options.issue: | 
| 588 patch_name = 'Issue %s' % options.issue | 591 patch_name = 'Issue %s' % options.issue | 
| 589 else: | 592 else: | 
| 590 options.name = 'Unnamed' | 593 options.name = 'Unnamed' | 
| 591 print('Note: use --name NAME to change the try job name.') | 594 print('Note: use --name NAME to change the try job name.') | 
| 592 if not options.email: | 595 if not options.email: | 
| 593 print('Warning: try job email will be sent to %s@google.com or ' | 596 print('Warning: TRYBOT_RESULTS_EMAIL_ADDRESS is not set. Try server ' | 
| 594 'something like that. Who knows? Set EMAIL_ADDRESS to override.' | 597 'results might\ngo to: %s@google.com.\n' % options.user) | 
| 595 % options.user) | 598 else: | 
| 599 print('Results will be emailed to: ' + options.email) | |
| 596 | 600 | 
| 597 # Send the patch. | 601 # Send the patch. | 
| 598 options.send_patch(options) | 602 options.send_patch(options) | 
| 599 print 'Patch \'%s\' sent to try server: %s' % (options.name, | 603 print 'Patch \'%s\' sent to try server: %s' % (options.name, | 
| 600 ', '.join(options.bot)) | 604 ', '.join(options.bot)) | 
| 601 except (InvalidScript, NoTryServerAccess), e: | 605 except (InvalidScript, NoTryServerAccess), e: | 
| 602 if swallow_exception: | 606 if swallow_exception: | 
| 603 return 1 | 607 return 1 | 
| 604 print e | 608 print e | 
| 605 return 1 | 609 return 1 | 
| 606 return 0 | 610 return 0 | 
| 607 | 611 | 
| 608 | 612 | 
| 609 if __name__ == "__main__": | 613 if __name__ == "__main__": | 
| 610 sys.exit(TryChange(None, None, False)) | 614 sys.exit(TryChange(None, None, False)) | 
| OLD | NEW |