Linux-SPDX Archive on lore.kernel.org
 help / color / Atom feed
From: Thomas Gleixner <tglx@linutronix.de>
To: J Lovejoy <opensource@jilayne.com>
Cc: Allison Randal <allison@lohutok.net>, linux-spdx@vger.kernel.org
Subject: Re: [patch 02/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 2
Date: Mon, 20 May 2019 19:47:51 +0200 (CEST)
Message-ID: <alpine.DEB.2.21.1905201938190.1635@nanos.tec.linutronix.de> (raw)
In-Reply-To: <F17CC104-2571-44A9-8115-0E129E3D06C4@jilayne.com>


[-- Attachment #1: Type: text/plain, Size: 1334 bytes --]

On Mon, 20 May 2019, J Lovejoy wrote:
> > On May 19, 2019, at 10:00 PM, Thomas Gleixner <tglx@linutronix.de> wrote:
> > 
> > 
> > On Sun, 19 May 2019, Allison Randal wrote:
> > 
> >> Since the text says "gnu *library* general public license", shouldn't
> >> the SPDX license identifier be LGPL instead of GPL?
> > 
> > Well spotted
> 
> indeed. is anyone else concerned that the scanner(s) didn’t catch this?
> This is the kind of thing that (I think) would have been
> caught. Considering that we have to rely on the tooling to a certain
> degree (i.e. we can’t possibly look at every file individually), this has
> me a bit worried…

The problem with the tools is that they need tons of heuristics to cope
with the endless amount of mess. So it's a given that some of the
heuristics are wrong.

I spent quite some time to think about a solution to this and the way I
proposed doing it, e.g. reduce the mess to normalized patterns, allows us
to avoid staring at 20k patch snippets.

We really need to spot the failures in the patterns, not the actual file
level patches. The patcher itself is going to remove the file level
equivalent of the normalized pattern and add the concluded SPDX identifier
instead. Not more, not less.

If the tools would be perfect we would not need to look at any of these
things ...

Thanks,

	tglx

  reply index

Thread overview: 148+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-19 13:51 [patch 00/25] First batch Thomas Gleixner
2019-05-19 13:51 ` [patch 01/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 1 Thomas Gleixner
2019-05-19 17:23   ` J Lovejoy
2019-05-19 17:28     ` Kate Stewart
2019-05-19 21:47       ` Allison Randal
2019-05-20  1:03   ` Steve Winslow
2019-05-20  9:00   ` Greg KH
2019-05-20  9:04     ` Thomas Gleixner
2019-05-20  9:22       ` Greg KH
2019-05-20 20:13         ` Thomas Gleixner
2019-05-21 14:18   ` Michael Ellerman
2019-05-19 13:51 ` [patch 02/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 2 Thomas Gleixner
2019-05-19 17:29   ` Kate Stewart
2019-05-19 21:50   ` Allison Randal
2019-05-20  0:44     ` Steve Winslow
2019-05-20  4:00     ` Thomas Gleixner
2019-05-20 17:17       ` J Lovejoy
2019-05-20 17:47         ` Thomas Gleixner [this message]
2019-05-20 18:09           ` Thomas Gleixner
2019-05-20 21:14           ` Allison Randal
2019-05-20 22:07     ` Philippe Ombredanne
2019-05-20 22:34       ` Allison Randal
2019-05-19 13:51 ` [patch 03/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 3 Thomas Gleixner
2019-05-19 17:24   ` J Lovejoy
2019-05-19 17:30     ` Kate Stewart
2019-05-19 21:51       ` Allison Randal
2019-05-20  1:04   ` Steve Winslow
2019-05-19 13:51 ` [patch 04/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 4 Thomas Gleixner
2019-05-19 17:25   ` J Lovejoy
2019-05-19 21:53     ` Allison Randal
2019-05-20  1:05   ` Steve Winslow
2019-05-19 13:51 ` [patch 05/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 5 Thomas Gleixner
2019-05-19 17:25   ` J Lovejoy
2019-05-19 21:54     ` Allison Randal
2019-05-20  1:05   ` Steve Winslow
2019-05-21  1:20   ` Kate Stewart
2019-05-19 13:51 ` [patch 06/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 6 Thomas Gleixner
2019-05-19 17:28   ` J Lovejoy
2019-05-19 22:32     ` Allison Randal
2019-05-20  0:53   ` Steve Winslow
2019-05-20 11:10     ` Thomas Gleixner
2019-05-20 16:50       ` Philippe Ombredanne
2019-05-19 13:51 ` [patch 07/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 7 Thomas Gleixner
2019-05-19 17:29   ` J Lovejoy
2019-05-19 22:35     ` Allison Randal
2019-05-20  1:06   ` Steve Winslow
2019-05-21  1:21   ` Kate Stewart
2019-05-19 13:51 ` [patch 08/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 8 Thomas Gleixner
2019-05-19 17:30   ` J Lovejoy
2019-05-19 22:40     ` Allison Randal
2019-05-20  0:57   ` Steve Winslow
2019-05-20  8:51     ` Thomas Gleixner
2019-05-19 13:51 ` [patch 09/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 9 Thomas Gleixner
2019-05-19 17:30   ` J Lovejoy
2019-05-19 23:17     ` Allison Randal
2019-05-20  1:06   ` Steve Winslow
2019-05-21  1:21   ` Kate Stewart
2019-05-19 13:51 ` [patch 10/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 10 Thomas Gleixner
2019-05-19 17:31   ` J Lovejoy
2019-05-19 23:18     ` Allison Randal
2019-05-20  1:06   ` Steve Winslow
2019-05-21  1:22   ` Kate Stewart
2019-05-19 13:51 ` [patch 11/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 11 Thomas Gleixner
2019-05-19 17:31   ` J Lovejoy
2019-05-19 23:19     ` Allison Randal
2019-05-20  1:07   ` Steve Winslow
2019-05-21  1:22   ` Kate Stewart
2019-05-19 13:51 ` [patch 12/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 12 Thomas Gleixner
2019-05-19 17:36   ` J Lovejoy
2019-05-19 23:26     ` Allison Randal
2019-05-20  1:07   ` Steve Winslow
2019-05-21  1:23   ` Kate Stewart
2019-05-19 13:51 ` [patch 13/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 13 Thomas Gleixner
2019-05-19 17:38   ` J Lovejoy
2019-05-19 23:27     ` Allison Randal
2019-05-20  1:08   ` Steve Winslow
2019-05-21  1:24   ` Kate Stewart
2019-05-21 14:20   ` Michael Ellerman
2019-05-19 13:51 ` [patch 14/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 14 Thomas Gleixner
     [not found]   ` <01BD65EE-6C09-45E4-ACAB-074B16C19EE5@jilayne.com>
2019-05-19 17:49     ` J Lovejoy
2019-05-19 23:30       ` Allison Randal
2019-05-20  1:08   ` Steve Winslow
2019-05-21  1:26   ` Kate Stewart
2019-05-19 13:51 ` [patch 15/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 15 Thomas Gleixner
     [not found]   ` <345D7893-767F-42C5-A73D-599338174AAA@jilayne.com>
2019-05-19 17:50     ` J Lovejoy
2019-05-19 23:40       ` Allison Randal
2019-05-20  1:09   ` Steve Winslow
2019-05-21  1:26   ` Kate Stewart
2019-05-19 13:51 ` [patch 16/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 16 Thomas Gleixner
2019-05-19 17:41   ` J Lovejoy
2019-05-19 23:40     ` Allison Randal
2019-05-20  0:27   ` Steve Winslow
2019-05-20  0:42     ` Richard Fontana
2019-05-20  2:42       ` Allison Randal
2019-05-20  5:25         ` Thomas Gleixner
2019-05-20 16:38       ` Philippe Ombredanne
2019-05-20 17:10         ` Richard Fontana
2019-05-20 17:28           ` Thomas Gleixner
2019-05-20 22:43             ` Philippe Ombredanne
2019-05-20  8:31   ` Thomas Gleixner
2019-05-19 13:51 ` [patch 17/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 17 Thomas Gleixner
2019-05-19 17:41   ` J Lovejoy
2019-05-19 23:41     ` Allison Randal
2019-05-20  1:10   ` Steve Winslow
2019-05-21  1:28   ` Kate Stewart
2019-05-19 13:51 ` [patch 18/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 18 Thomas Gleixner
2019-05-19 17:42   ` J Lovejoy
2019-05-19 23:49     ` Allison Randal
2019-05-20  1:10   ` Steve Winslow
2019-05-21  1:28   ` Kate Stewart
2019-05-19 13:51 ` [patch 19/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 19 Thomas Gleixner
2019-05-19 17:50   ` J Lovejoy
2019-05-19 23:50     ` Allison Randal
2019-05-20  1:11   ` Steve Winslow
2019-05-21  1:29   ` Kate Stewart
2019-05-19 13:51 ` [patch 20/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 20 Thomas Gleixner
2019-05-19 17:43   ` J Lovejoy
2019-05-19 23:57     ` Allison Randal
2019-05-20  1:11   ` Steve Winslow
2019-05-21  1:30   ` Kate Stewart
2019-05-19 13:51 ` [patch 21/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 21 Thomas Gleixner
2019-05-19 17:44   ` J Lovejoy
2019-05-19 23:58     ` Allison Randal
2019-05-20  1:11   ` Steve Winslow
2019-05-21  1:31   ` Kate Stewart
2019-05-19 13:51 ` [patch 22/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 22 Thomas Gleixner
2019-05-19 17:45   ` J Lovejoy
2019-05-20  1:26     ` Allison Randal
2019-05-20  1:11   ` Steve Winslow
2019-05-21  1:34   ` Kate Stewart
2019-05-21  5:46     ` Thomas Gleixner
2019-05-19 13:51 ` [patch 23/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 23 Thomas Gleixner
2019-05-19 17:45   ` J Lovejoy
2019-05-20  1:27     ` Allison Randal
2019-05-20  1:12   ` Steve Winslow
2019-05-21  1:35   ` Kate Stewart
2019-05-19 13:51 ` [patch 24/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 24 Thomas Gleixner
2019-05-19 17:51   ` J Lovejoy
2019-05-20  1:29     ` Allison Randal
2019-05-20  1:15   ` Steve Winslow
2019-05-21  1:36   ` Kate Stewart
2019-05-19 13:51 ` [patch 25/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 25 Thomas Gleixner
2019-05-19 17:52   ` J Lovejoy
2019-05-20  1:31     ` Allison Randal
2019-05-20  1:15   ` Steve Winslow
2019-05-21  1:37   ` Kate Stewart
2019-05-21 10:20 ` [patch 00/25] First batch Greg KH
2019-05-21 10:24   ` Thomas Gleixner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.DEB.2.21.1905201938190.1635@nanos.tec.linutronix.de \
    --to=tglx@linutronix.de \
    --cc=allison@lohutok.net \
    --cc=linux-spdx@vger.kernel.org \
    --cc=opensource@jilayne.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-SPDX Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-spdx/0 linux-spdx/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-spdx linux-spdx/ https://lore.kernel.org/linux-spdx \
		linux-spdx@vger.kernel.org
	public-inbox-index linux-spdx

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-spdx


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git