Issue 4028 - Add support for Edge extensions to buildtools

build.py

 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 import os
 import sys
 import re
 import subprocess
 import shutil
 import buildtools
@@ skipping 160 matching lines @@
             'name': command.name,
             'params': command.params,
             'description': description,
             'options': '\n'.join(options)
         }
 
 
 def runBuild(baseDir, scriptName, opts, args, type):
     kwargs = {}
     for option, value in opts:
-        if option in ('-l', '--locales') and type == 'gecko':

[Sebastian Noack, 2016/10/04 19:58:04]

While refactoring this code anyway, this should be a set rather than a tuple,
same below.

[Vasily Kuznetsov, 2016/10/11 15:58:27]

Done.

+        if option in {'-l', '--locales'} and type == 'gecko':
             kwargs['locales'] = value.split(',')
-        elif option in ('-b', '--build'):
+        elif option in {'-b', '--build'}:
             kwargs['buildNum'] = value
             if type != 'gecko' and not kwargs['buildNum'].isdigit():
                 raise TypeError('Build number must be numerical')
-        elif option in ('-k', '--key'):
+        elif option in {'-k', '--key'}:
             kwargs['keyFile'] = value
-        elif option in ('-m', '--multi-compartment') and type == 'gecko':
+        elif option in {'-m', '--multi-compartment'} and type == 'gecko':
             kwargs['multicompartment'] = True
-        elif option in ('-r', '--release'):
+        elif option in {'-r', '--release'}:
             kwargs['releaseBuild'] = True
     if len(args) > 0:
         kwargs['outFile'] = args[0]
 
     if type == 'gecko':
         import buildtools.packagerGecko as packager
     elif type == 'chrome':
         import buildtools.packagerChrome as packager
     elif type == 'safari':
         import buildtools.packagerSafari as packager
     elif type == 'edge':
         import buildtools.packagerEdge as packager
 
     packager.createBuild(baseDir, type=type, **kwargs)

[Sebastian Noack, 2016/10/04 19:58:05]

Since you are apparently changing the calling code anyway, how about renaming
createBuild() to create_build() in existing modules, rather than adding #noqa in
the new code?

[Vasily Kuznetsov, 2016/10/11 15:58:27]

That's a lot of unrelated changes to pull in. Also `releaseAutomation.py` calls
`createBuild` and god knows what else outside of buildtools. I would prefer to
make a separate commit for these kinds of renames.

[Sebastian Noack, 2016/10/12 11:07:48]

I was only talking about packagerEdge.create_build() which isn't used outside of
buildtools yet, since it doesn't exist before these changes. Sure, this would be
inconsistent (for now) but might be better than introducing new names using
camalcase in new code.

[Vasily Kuznetsov, 2016/10/12 14:13:42]

I tried doing it, but unfortunately we still can't remove the #noqa. There are
two other naming issues in the definition of `packagerEdge.createBuild`: one
argument is called `type` (that redefines a builtin) and all other arguments are
in camelcase. The first issue is easy to address by removing `type` from the
signature and adding a `**kw` (ugly but works) but the second one would require
either putting just `*args, **kw` for arguments and then reconstructing them
manually or calling `packagerEdge.create_build` in a different way from all
other `createBuild` functions. I don't like either of these options so this
leaves us with the choice between renaming `packagerEdge.createBuild` to
`create_build` and still having the #noqa or just leaving things as they are. It
seems like renaming is not really worth it. What do you think?

[Sebastian Noack, 2016/10/12 14:43:26]

I guess, let's leave it as is for now. We still should probably change the
calling convention for all packagers in one go with a different issue.

[Vasily Kuznetsov, 2016/10/13 11:56:35]

Fully agree.

 
 
 def runAutoInstall(baseDir, scriptName, opts, args, type):
     if len(args) == 0:
         print 'Port of the Extension Auto-Installer needs to be specified'
         usage(scriptName, type, 'autoinstall')
         return
 
     multicompartment = False
     for option, value in opts:
@@ skipping 382 matching lines @@
                 if option in ('-h', '--help'):
                     usage(scriptName, type, command)
                     sys.exit()
             commands[command](baseDir, scriptName, opts, args, type)
         else:
             print 'Command %s is not supported for this application type' % command
             usage(scriptName, type)
     else:
         print 'Command %s is unrecognized' % command
         usage(scriptName, type)