Re: [patch V2 7/7] scripts: Add SPDX checker script
From: Geert Uytterhoeven
Date: Tue Aug 28 2018 - 05:19:46 EST
On Mon, May 14, 2018 at 4:38 PM Philippe Ombredanne
<pombredanne@xxxxxxxx> wrote:
> On Wed, Apr 25, 2018 at 10:30 PM, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
> > The SPDX-License-Identifiers are growing in the kernel and so grow
> > expression failures and license IDs are used which have no corresponding
> > license text file in the LICENSES directory.
> >
> > Add a script which gathers information from the LICENSES directory,
> > i.e. the various tags in the licenses and exception files and then scans
> > either input from stdin, which it treats as a single file or if started
> > without arguments it scans the full kernel tree.
> >
> > It checks whether the license expression syntax is correct and also
> > validates whether the license identifiers used in the expressions are
> > available in the LICENSES files.
>
> Looking good to me! And the use of ply is sleek.
For the record (currently Google doesn't find this issue yet):
ImportError: No module named ply
$ sudo apt-get install python-ply
ImportError: No module named git
$ sudo apt-get install python-git
Works!
Apparently I didn't have any of these two packages installed on any of my
Ubuntu 18.04LTS and 16.04LTS machines...
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds