[OE-core] [PATCH 08/12] recipetool: npm: Do not add package.json files to LIC_FILES_CHKSUM

2021-10-08 Thread Stefan Herbrechtsmeier
From: Stefan Herbrechtsmeier 

The package.json files doesn't contain any licenses. The name of the
license doesn't comply the license requirements of most liceneses.

Signed-off-by: Stefan Herbrechtsmeier 
---

(no changes since v1)

 scripts/lib/recipetool/create_npm.py | 10 --
 1 file changed, 10 deletions(-)

diff --git a/scripts/lib/recipetool/create_npm.py 
b/scripts/lib/recipetool/create_npm.py
index 2bcae91dfa..c939780931 100644
--- a/scripts/lib/recipetool/create_npm.py
+++ b/scripts/lib/recipetool/create_npm.py
@@ -115,17 +115,9 @@ class NpmRecipeHandler(RecipeHandler):
 
 def _handle_licenses(self, srctree, shrinkwrap_file, dev):
 """Return the extra license files and the list of packages"""
-licfiles = []
 packages = {}
 
-def _licfiles_append(licfile):
-"""Append 'licfile' to the license files list"""
-licfilepath = os.path.join(srctree, licfile)
-licmd5 = bb.utils.md5_file(licfilepath)
-licfiles.append("file://%s;md5=%s" % (licfile, licmd5))
-
 # Handle the parent package
-_licfiles_append("package.json")
 packages["${PN}"] = ""
 
 # Handle the dependencies
@@ -133,7 +125,6 @@ class NpmRecipeHandler(RecipeHandler):
 suffix = "-".join([self._npm_name(dep) for dep in deptree])
 destdirs = [os.path.join("node_modules", dep) for dep in deptree]
 destdir = os.path.join(*destdirs)
-_licfiles_append(os.path.join(destdir, "package.json"))
 packages["${PN}-" + suffix] = destdir
 
 with open(shrinkwrap_file, "r") as f:
@@ -246,7 +237,6 @@ class NpmRecipeHandler(RecipeHandler):
 
 bb.note("Handling licences ...")
 (licfiles, packages) = self._handle_licenses(srctree, shrinkwrap_file, 
dev)
-extravalues["LIC_FILES_CHKSUM"] = licfiles
 split_pkg_licenses(guess_license(srctree, d), packages, lines_after, 
[])
 
 classes.append("npm")
-- 
2.20.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156751): 
https://lists.openembedded.org/g/openembedded-core/message/156751
Mute This Topic: https://lists.openembedded.org/mt/86165228/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 08/12] recipetool: npm: Do not add package.json files to LIC_FILES_CHKSUM

2021-10-07 Thread Stefan Herbrechtsmeier
From: Stefan Herbrechtsmeier 

The package.json files doesn't contain any licenses. The name of the
license doesn't comply the license requirements of most liceneses.

Signed-off-by: Stefan Herbrechtsmeier 
---

 scripts/lib/recipetool/create_npm.py | 10 --
 1 file changed, 10 deletions(-)

diff --git a/scripts/lib/recipetool/create_npm.py 
b/scripts/lib/recipetool/create_npm.py
index 2bcae91dfa..c939780931 100644
--- a/scripts/lib/recipetool/create_npm.py
+++ b/scripts/lib/recipetool/create_npm.py
@@ -115,17 +115,9 @@ class NpmRecipeHandler(RecipeHandler):
 
 def _handle_licenses(self, srctree, shrinkwrap_file, dev):
 """Return the extra license files and the list of packages"""
-licfiles = []
 packages = {}
 
-def _licfiles_append(licfile):
-"""Append 'licfile' to the license files list"""
-licfilepath = os.path.join(srctree, licfile)
-licmd5 = bb.utils.md5_file(licfilepath)
-licfiles.append("file://%s;md5=%s" % (licfile, licmd5))
-
 # Handle the parent package
-_licfiles_append("package.json")
 packages["${PN}"] = ""
 
 # Handle the dependencies
@@ -133,7 +125,6 @@ class NpmRecipeHandler(RecipeHandler):
 suffix = "-".join([self._npm_name(dep) for dep in deptree])
 destdirs = [os.path.join("node_modules", dep) for dep in deptree]
 destdir = os.path.join(*destdirs)
-_licfiles_append(os.path.join(destdir, "package.json"))
 packages["${PN}-" + suffix] = destdir
 
 with open(shrinkwrap_file, "r") as f:
@@ -246,7 +237,6 @@ class NpmRecipeHandler(RecipeHandler):
 
 bb.note("Handling licences ...")
 (licfiles, packages) = self._handle_licenses(srctree, shrinkwrap_file, 
dev)
-extravalues["LIC_FILES_CHKSUM"] = licfiles
 split_pkg_licenses(guess_license(srctree, d), packages, lines_after, 
[])
 
 classes.append("npm")
-- 
2.20.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156715): 
https://lists.openembedded.org/g/openembedded-core/message/156715
Mute This Topic: https://lists.openembedded.org/mt/86141419/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-