Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: build.py

Issue 29345279: Noissue - Adapt quotes for compliance with our coding style in buildtools (Closed)
Patch Set: Created May 29, 2016, 1:27 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chainedconfigparser.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build.py
===================================================================
--- a/build.py
+++ b/build.py
@@ -446,14 +446,14 @@
usage(scriptName, type, 'release')
return
- if type == "gecko" and len(keyFiles) == 0:
- print >>sys.stderr, "Warning: no key file specified, creating an unsigned release build\n"
- elif type == "gecko" and len(keyFiles) > 1:
- print >>sys.stderr, "Error: too many key files, only one required"
+ if type == 'gecko' and len(keyFiles) == 0:
+ print >>sys.stderr, 'Warning: no key file specified, creating an unsigned release build\n'
+ elif type == 'gecko' and len(keyFiles) > 1:
+ print >>sys.stderr, 'Error: too many key files, only one required'
usage(scriptName, type, 'release')
return
- elif type == "chrome" and len(keyFiles) != 2:
- print >>sys.stderr, "Error: wrong number of key files specified, two keys (Chrome and Safari) required for the release"
+ elif type == 'chrome' and len(keyFiles) != 2:
+ print >>sys.stderr, 'Error: wrong number of key files specified, two keys (Chrome and Safari) required for the release'
usage(scriptName, type, 'release')
return
@@ -531,10 +531,7 @@
with addCommand(runReleaseAutomation, 'release') as command:
command.shortDescription = 'Run release automation'
- command.description = 'Note: If you are not the project owner then you '\
- 'probably don\'t want to run this!\n\n'\
- 'Runs release automation: creates downloads for the new version, tags '\
- 'source code repository as well as downloads and buildtools repository.'
+ command.description = 'Note: If you are not the project owner then you ' "probably don't want to run this!\n\n" 'Runs release automation: creates downloads for the new version, tags ' 'source code repository as well as downloads and buildtools repository.'
command.addOption('File containing private key and certificates required to sign the release. Note that for Chrome releases this option needs to be specified twice: first a key to sign Chrome builds, then another to sign the Safari build.', short='k', long='key', value='file', types=('gecko', 'chrome'))
command.addOption('Directory containing downloads repository (if omitted ../downloads is assumed)', short='d', long='downloads', value='dir')
command.params = '[options] <version>'
« no previous file with comments | « no previous file | chainedconfigparser.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld