| Index: ensure_dependencies.py | 
| =================================================================== | 
| --- a/ensure_dependencies.py | 
| +++ b/ensure_dependencies.py | 
| @@ -9,16 +9,17 @@ | 
| import posixpath | 
| import re | 
| import io | 
| import errno | 
| import logging | 
| import subprocess | 
| import urlparse | 
| import argparse | 
| +import json | 
|  | 
| from collections import OrderedDict | 
| from ConfigParser import RawConfigParser | 
|  | 
| USAGE = ''' | 
| A dependencies file should look like this: | 
|  | 
| # VCS-specific root URLs for the repositories | 
| @@ -38,18 +39,20 @@ | 
| # from a specific Git repository, specifying the revision ID. | 
| adblockpluschrome = git:git@github.com:user/adblockpluschrome.git@1fad3a7 | 
| ''' | 
|  | 
| SKIP_DEPENDENCY_UPDATES = os.environ.get( | 
| 'SKIP_DEPENDENCY_UPDATES', '' | 
| ).lower() not in ('', '0', 'false') | 
|  | 
| +NPM_LOCKFILE = '.npm_install_lock' | 
|  | 
| -class Mercurial(): | 
| + | 
| +class Mercurial: | 
| def istype(self, repodir): | 
| return os.path.exists(os.path.join(repodir, '.hg')) | 
|  | 
| def clone(self, source, target): | 
| if not source.endswith('/'): | 
| source += '/' | 
| subprocess.check_call(['hg', 'clone', '--quiet', '--noupdate', source, target]) | 
|  | 
| @@ -86,17 +89,17 @@ | 
|  | 
| module = os.path.relpath(target, repo) | 
| _ensure_line_exists(ignore_path, module) | 
|  | 
| def postprocess_url(self, url): | 
| return url | 
|  | 
|  | 
| -class Git(): | 
| +class Git: | 
| def istype(self, repodir): | 
| return os.path.exists(os.path.join(repodir, '.git')) | 
|  | 
| def clone(self, source, target): | 
| source = source.rstrip('/') | 
| if not source.endswith('.git'): | 
| source += '.git' | 
| subprocess.check_call(['git', 'clone', '--quiet', source, target]) | 
| @@ -130,16 +133,17 @@ | 
| _ensure_line_exists(exclude_file, module) | 
|  | 
| def postprocess_url(self, url): | 
| # Handle alternative syntax of SSH URLS | 
| if '@' in url and ':' in url and not urlparse.urlsplit(url).scheme: | 
| return 'ssh://' + url.replace(':', '/', 1) | 
| return url | 
|  | 
| + | 
| repo_types = OrderedDict(( | 
| ('hg', Mercurial()), | 
| ('git', Git()), | 
| )) | 
|  | 
| # [vcs:]value | 
| item_regexp = re.compile( | 
| '^(?:(' + '|'.join(map(re.escape, repo_types.keys())) + '):)?' | 
| @@ -240,74 +244,132 @@ | 
|  | 
| def get_repo_type(repo): | 
| for name, repotype in repo_types.iteritems(): | 
| if repotype.istype(repo): | 
| return name | 
| return 'hg' | 
|  | 
|  | 
| +def resolve_npm_dependencies(target, vcs): | 
| +    """Install Node.js production-only dependencies if necessary and desired. | 
| + | 
| +    When the target dependency has additional Node.js dependencies declared | 
| +    run "npm install --only=production --loglevel=warn" to resolve the declared | 
| +    dependencies. | 
| + | 
| +    Additionally, make sure that any VCS will ignore the installed files. | 
| + | 
| +    Requires Node.js to be installed locally. | 
| +    """ | 
| +    try: | 
| +        with open(os.path.join(target, 'package.json'), 'r') as fp: | 
| +            package_data = json.load(fp) | 
| + | 
| +        # In case a package.json does not exist at all or if there are no | 
| +        # production dependencies declared, we don't need to run npm and can | 
| +        # bail out early. | 
| +        if not package_data.get('dependencies', False): | 
| +            return | 
| +    except IOError: | 
| +        return | 
| + | 
| +    try: | 
| +        # Create an empty file, which gets deleted after successfully | 
| +        # installing Node.js dependencies. | 
| +        lockfile_path = os.path.join(target, NPM_LOCKFILE) | 
| +        open(lockfile_path, 'a').close() | 
| + | 
| +        if os.name == 'nt': | 
| +            # Windows' CreateProcess() (called by subprocess.Popen()) only | 
| +            # resolves executables ending in .exe. The windows installation of | 
| +            # Node.js only provides a npm.cmd, which is executable but won't | 
| +            # be recognized as such by CreateProcess(). | 
| +            npm_exec = 'npm.cmd' | 
| +        else: | 
| +            npm_exec = 'npm' | 
| + | 
| +        cmd = [npm_exec, 'install', '--only=production', '--loglevel=warn', | 
| +               '--no-package-lock', '--no-optional'] | 
| +        subprocess.check_output(cmd, cwd=target) | 
| + | 
| +        repo_types[vcs].ignore(os.path.join(target, NPM_LOCKFILE), target) | 
| +        repo_types[vcs].ignore(os.path.join(target, 'node_modules'), target) | 
| + | 
| +        os.remove(lockfile_path) | 
| +    except OSError as e: | 
| +        import errno | 
| +        if e.errno == errno.ENOENT: | 
| +            logging.error('Failed to install Node.js dependencies for %s,' | 
| +                          ' please ensure Node.js is installed.', target) | 
| +        else: | 
| +            raise | 
| + | 
| + | 
| def ensure_repo(parentrepo, parenttype, target, type, root, sourcename): | 
| if os.path.exists(target): | 
| -        return | 
| +        return False | 
|  | 
| if SKIP_DEPENDENCY_UPDATES: | 
| logging.warning('SKIP_DEPENDENCY_UPDATES environment variable set, ' | 
| '%s not cloned', target) | 
| -        return | 
| +        return False | 
|  | 
| postprocess_url = repo_types[type].postprocess_url | 
| root = postprocess_url(root) | 
| sourcename = postprocess_url(sourcename) | 
|  | 
| if os.path.exists(root): | 
| url = os.path.join(root, sourcename) | 
| else: | 
| url = urlparse.urljoin(root, sourcename) | 
|  | 
| logging.info('Cloning repository %s into %s' % (url, target)) | 
| repo_types[type].clone(url, target) | 
| repo_types[parenttype].ignore(target, parentrepo) | 
| +    return True | 
|  | 
|  | 
| def update_repo(target, type, revision): | 
| resolved_revision = repo_types[type].get_revision_id(target, revision) | 
| current_revision = repo_types[type].get_revision_id(target) | 
|  | 
| if resolved_revision != current_revision: | 
| if SKIP_DEPENDENCY_UPDATES: | 
| logging.warning('SKIP_DEPENDENCY_UPDATES environment variable set, ' | 
| '%s not checked out to %s', target, revision) | 
| -            return | 
| +            return False | 
|  | 
| if not resolved_revision: | 
| logging.info('Revision %s is unknown, downloading remote changes' % revision) | 
| repo_types[type].pull(target) | 
| resolved_revision = repo_types[type].get_revision_id(target, revision) | 
| if not resolved_revision: | 
| raise Exception('Failed to resolve revision %s' % revision) | 
|  | 
| logging.info('Updating repository %s to revision %s' % (target, resolved_revision)) | 
| repo_types[type].update(target, resolved_revision, revision) | 
| +        return True | 
| +    return False | 
|  | 
|  | 
| def resolve_deps(repodir, level=0, self_update=True, overrideroots=None, skipdependencies=set()): | 
| config = read_deps(repodir) | 
| if config is None: | 
| if level == 0: | 
| logging.warning('No dependencies file in directory %s, nothing to do...\n%s' % (repodir, USAGE)) | 
| return | 
| if level >= 10: | 
| logging.warning('Too much subrepository nesting, ignoring %s' % repo) | 
| return | 
|  | 
| if overrideroots is not None: | 
| config['_root'] = overrideroots | 
|  | 
| -    for dir, sources in config.iteritems(): | 
| +    for dir, sources in sorted(config.iteritems()): | 
| if (dir.startswith('_') or | 
| skipdependencies.intersection([s[0] for s in sources if s[0]])): | 
| continue | 
|  | 
| target = safe_join(repodir, dir) | 
| parenttype = get_repo_type(repodir) | 
| _root = config.get('_root', {}) | 
|  | 
| @@ -315,18 +377,22 @@ | 
| if key == parenttype or key is None and vcs != '*': | 
| vcs = key | 
| source, rev = merge_seqs(sources.get('*'), sources.get(vcs)) | 
|  | 
| if not (vcs and source and rev): | 
| logging.warning('No valid source / revision found to create %s' % target) | 
| continue | 
|  | 
| -        ensure_repo(repodir, parenttype, target, vcs, _root.get(vcs, ''), source) | 
| -        update_repo(target, vcs, rev) | 
| +        repo_cloned = ensure_repo(repodir, parenttype, target, vcs, | 
| +                                  _root.get(vcs, ''), source) | 
| +        repo_updated = update_repo(target, vcs, rev) | 
| +        recent_npm_failed = os.path.exists(os.path.join(target, NPM_LOCKFILE)) | 
| +        if repo_cloned or repo_updated or recent_npm_failed: | 
| +            resolve_npm_dependencies(target, vcs) | 
| resolve_deps(target, level + 1, self_update=False, | 
| overrideroots=overrideroots, skipdependencies=skipdependencies) | 
|  | 
| if self_update and '_self' in config and '*' in config['_self']: | 
| source = safe_join(repodir, config['_self']['*']) | 
| try: | 
| with io.open(source, 'rb') as handle: | 
| sourcedata = handle.read() | 
| @@ -357,16 +423,17 @@ | 
| file_content = [l.strip() for l in f.readlines()] | 
| if not pattern in file_content: | 
| file_content.append(pattern) | 
| f.seek(0, os.SEEK_SET) | 
| f.truncate() | 
| for l in file_content: | 
| print >>f, l | 
|  | 
| + | 
| if __name__ == '__main__': | 
| logging.basicConfig(format='%(levelname)s: %(message)s', level=logging.INFO) | 
|  | 
| parser = argparse.ArgumentParser(description='Verify dependencies for a set of repositories, by default the repository of this script.') | 
| parser.add_argument('repos', metavar='repository', type=str, nargs='*', help='Repository path') | 
| parser.add_argument('-q', '--quiet', action='store_true', help='Suppress informational output') | 
| args = parser.parse_args() | 
|  | 
|  |