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

Unified Diff: build.py

Issue 9257092: More build tools improvements (Closed)
Patch Set: Added some more changes required to build Firefox and Chrome extensions from the same repository (s… Created Jan. 25, 2013, 1:47 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
@@ -400,26 +400,16 @@ def runReleaseAutomation(baseDir, script
return
if keyFile == None:
print 'Warning: no key file specified, creating an unsigned release build\n'
import buildtools.releaseAutomationGecko as releaseAutomation
releaseAutomation.run(baseDir, version, keyFile, downloadsRepo)
-def syncLocales(baseDir, scriptName, opts, args, type):
- if len(args) == 0:
- print 'Please specify the directory of the source Firefox extension as a parameter'
- usage(scriptName, type, 'synclocales')
- return
- sourceDir = args[0]
-
- import buildtools.localeSyncChrome as localeSync
- localeSync.run(baseDir, sourceDir)
-
def updatePSL(baseDir, scriptName, opts, args, type):
import buildtools.publicSuffixListUpdater as publicSuffixListUpdater
publicSuffixListUpdater.updatePSL(baseDir)
with addCommand(lambda baseDir, scriptName, opts, args, type: usage(scriptName, type), ('help', '-h', '--help')) as command:
command.shortDescription = 'Show this message'
with addCommand(runBuild, 'build') as command:
@@ -490,22 +480,16 @@ with addCommand(runReleaseAutomation, 'r
'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', short='k', long='key', value='file', types=('gecko'))
command.addOption('Directory containing downloads repository (if omitted ../downloads is assumed)', short='d', long='downloads', value='dir')
command.params = '[options] <version>'
command.supportedTypes = ('gecko')
-with addCommand(syncLocales, 'synclocales') as command:
- command.shortDescription = 'Sync locales with a Firefox extension'
- command.description = 'Updates locale files with strings from a Firefox extension corresponding to the entries in [locale_sync] metadata section.'
- command.params = '<firefox_addon_directory>'
- command.supportedTypes = ('chrome')
-
with addCommand(updatePSL, 'updatepsl') as command:
command.shortDescription = 'Updates Public Suffix List'
command.description = 'Downloads Public Suffix List (see http://publicsuffix.org/) and generates lib/publicSuffixList.js from it.'
command.supportedTypes = ('chrome')
def getType(baseDir, scriptName, args):
# Look for an explicit type parameter (has to be the first parameter)
if len(args) >= 2 and args[0] == '-t':
« no previous file with comments | « no previous file | chainedconfigparser.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld