diff options
Diffstat (limited to 'scripts/lib/recipetool/create_npm.py')
| -rw-r--r-- | scripts/lib/recipetool/create_npm.py | 57 |
1 files changed, 54 insertions, 3 deletions
diff --git a/scripts/lib/recipetool/create_npm.py b/scripts/lib/recipetool/create_npm.py index 0e33cc9a1e..4bf6caed5c 100644 --- a/scripts/lib/recipetool/create_npm.py +++ b/scripts/lib/recipetool/create_npm.py | |||
| @@ -17,20 +17,37 @@ | |||
| 17 | 17 | ||
| 18 | import logging | 18 | import logging |
| 19 | import json | 19 | import json |
| 20 | from recipetool.create import RecipeHandler | 20 | from recipetool.create import RecipeHandler, split_pkg_licenses |
| 21 | 21 | ||
| 22 | logger = logging.getLogger('recipetool') | 22 | logger = logging.getLogger('recipetool') |
| 23 | 23 | ||
| 24 | 24 | ||
| 25 | class NpmRecipeHandler(RecipeHandler): | 25 | class NpmRecipeHandler(RecipeHandler): |
| 26 | def _handle_license(self, data): | ||
| 27 | ''' | ||
| 28 | Handle the license value from an npm package.json file | ||
| 29 | ''' | ||
| 30 | license = None | ||
| 31 | if 'license' in data: | ||
| 32 | license = data['license'] | ||
| 33 | if isinstance(license, dict): | ||
| 34 | license = license.get('type', None) | ||
| 35 | return None | ||
| 36 | |||
| 26 | def process(self, srctree, classes, lines_before, lines_after, handled, extravalues): | 37 | def process(self, srctree, classes, lines_before, lines_after, handled, extravalues): |
| 38 | import oe | ||
| 39 | from collections import OrderedDict | ||
| 40 | |||
| 27 | if 'buildsystem' in handled: | 41 | if 'buildsystem' in handled: |
| 28 | return False | 42 | return False |
| 29 | 43 | ||
| 44 | def read_package_json(fn): | ||
| 45 | with open(fn, 'r') as f: | ||
| 46 | return json.loads(f.read()) | ||
| 47 | |||
| 30 | files = RecipeHandler.checkfiles(srctree, ['package.json']) | 48 | files = RecipeHandler.checkfiles(srctree, ['package.json']) |
| 31 | if files: | 49 | if files: |
| 32 | with open(files[0], 'r') as f: | 50 | data = read_package_json(files[0]) |
| 33 | data = json.loads(f.read()) | ||
| 34 | if 'name' in data and 'version' in data: | 51 | if 'name' in data and 'version' in data: |
| 35 | extravalues['PN'] = data['name'] | 52 | extravalues['PN'] = data['name'] |
| 36 | extravalues['PV'] = data['version'] | 53 | extravalues['PV'] = data['version'] |
| @@ -40,6 +57,40 @@ class NpmRecipeHandler(RecipeHandler): | |||
| 40 | lines_before.append('SUMMARY = "%s"' % data['description']) | 57 | lines_before.append('SUMMARY = "%s"' % data['description']) |
| 41 | if 'homepage' in data: | 58 | if 'homepage' in data: |
| 42 | lines_before.append('HOMEPAGE = "%s"' % data['homepage']) | 59 | lines_before.append('HOMEPAGE = "%s"' % data['homepage']) |
| 60 | |||
| 61 | # Split each npm module out to is own package | ||
| 62 | npmpackages = oe.package.npm_split_package_dirs(srctree) | ||
| 63 | for item in handled: | ||
| 64 | if isinstance(item, tuple): | ||
| 65 | if item[0] == 'license': | ||
| 66 | licvalues = item[1] | ||
| 67 | break | ||
| 68 | if licvalues: | ||
| 69 | # Augment the license list with information we have in the packages | ||
| 70 | licenses = {} | ||
| 71 | license = self._handle_license(data) | ||
| 72 | if license: | ||
| 73 | licenses['${PN}'] = license | ||
| 74 | for pkgname, pkgitem in npmpackages.iteritems(): | ||
| 75 | _, pdata = pkgitem | ||
| 76 | license = self._handle_license(pdata) | ||
| 77 | if license: | ||
| 78 | licenses[pkgname] = license | ||
| 79 | # Now write out the package-specific license values | ||
| 80 | # We need to strip out the json data dicts for this since split_pkg_licenses | ||
| 81 | # isn't expecting it | ||
| 82 | packages = OrderedDict((x,y[0]) for x,y in npmpackages.iteritems()) | ||
| 83 | packages['${PN}'] = '' | ||
| 84 | pkglicenses = split_pkg_licenses(licvalues, packages, lines_after, licenses) | ||
| 85 | all_licenses = list(set([item for pkglicense in pkglicenses.values() for item in pkglicense])) | ||
| 86 | # Go back and update the LICENSE value since we have a bit more | ||
| 87 | # information than when that was written out (and we know all apply | ||
| 88 | # vs. there being a choice, so we can join them with &) | ||
| 89 | for i, line in enumerate(lines_before): | ||
| 90 | if line.startswith('LICENSE = '): | ||
| 91 | lines_before[i] = 'LICENSE = "%s"' % ' & '.join(all_licenses) | ||
| 92 | break | ||
| 93 | |||
| 43 | return True | 94 | return True |
| 44 | 95 | ||
| 45 | return False | 96 | return False |
