2019-05-11 20:31:48

by Sven Eckelmann

[permalink] [raw]
Subject: [PATCH] scripts/spdxcheck.py: Fix path to deprecated licenses

The directory name for other licenses was changed to "deprecated" in
commit 62be257e986d ("LICENSES: Rename other to deprecated"). But it was
not changed for spdxcheck.py. As result, checkpatch failed with

FAIL: "Blob or Tree named 'other' not found"
Traceback (most recent call last):
File "scripts/spdxcheck.py", line 240, in <module>
spdx = read_spdxdata(repo)
File "scripts/spdxcheck.py", line 41, in read_spdxdata
for el in lictree[d].traverse():
File "/usr/lib/python2.7/dist-packages/git/objects/tree.py", line 298, in __getitem__
return self.join(item)
File "/usr/lib/python2.7/dist-packages/git/objects/tree.py", line 244, in join
raise KeyError(msg % file)
KeyError: "Blob or Tree named 'other' not found"

Cc: Jonathan Corbet <[email protected]>
Cc: Christoph Hellwig <[email protected]>
Fixes: 62be257e986d ("LICENSES: Rename other to deprecated")
Signed-off-by: Sven Eckelmann <[email protected]>
---
scripts/spdxcheck.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/spdxcheck.py b/scripts/spdxcheck.py
index 4fe392e507fb..1a39b34588b7 100755
--- a/scripts/spdxcheck.py
+++ b/scripts/spdxcheck.py
@@ -32,7 +32,7 @@ class SPDXdata(object):
def read_spdxdata(repo):

# The subdirectories of LICENSES in the kernel source
- license_dirs = [ "preferred", "other", "exceptions" ]
+ license_dirs = [ "preferred", "deprecated", "exceptions" ]
lictree = repo.head.commit.tree['LICENSES']

spdx = SPDXdata()
--
2.20.1


2019-05-13 07:03:29

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH] scripts/spdxcheck.py: Fix path to deprecated licenses

For both patches:

Reviewed-by: Christoph Hellwig <[email protected]>

2019-05-20 19:29:12

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH] scripts/spdxcheck.py: Fix path to deprecated licenses

On Sat, 11 May 2019 22:19:16 +0200
Sven Eckelmann <[email protected]> wrote:

> The directory name for other licenses was changed to "deprecated" in
> commit 62be257e986d ("LICENSES: Rename other to deprecated"). But it was
> not changed for spdxcheck.py. As result, checkpatch failed with

Both patches applied, thanks.

jon