Linux-SPDX Archive on lore.kernel.org
 help / color / Atom feed
From: Richard Fontana <rfontana@redhat.com>
To: Philippe Ombredanne <pombredanne@nexb.com>
Cc: linux-spdx@vger.kernel.org, Steve Winslow <swinslow@linuxfoundation.org>
Subject: Re: [patch 16/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 16
Date: Mon, 20 May 2019 13:10:28 -0400
Message-ID: <CAC1cPGzUmbOrYZP9wth4fc639Kan+mXqrzB0DbNLofskVxYMJw@mail.gmail.com> (raw)
In-Reply-To: <CAOFm3uHrNicnK5ydn_Pv0Q+R+DcK0+0mGG6RcxXUZS_USc551A@mail.gmail.com>

On Mon, May 20, 2019 at 12:39 PM Philippe Ombredanne
<pombredanne@nexb.com> wrote:
>
> Richard, Steve:
>
> On Mon, May 20, 2019 at 2:42 AM Richard Fontana <rfontana@redhat.com> wrote:
> >
> > On Sun, May 19, 2019 at 8:27 PM Steve Winslow
> > <swinslow@linuxfoundation.org> wrote:
> > >
> > > For this one, the additional disclaimer paragraph at the bottom
> > > (beginning with "THIS SOFTWARE IS PROVIDED BY...") appears to come
> > > from a BSD-style license, not GPL. I'm not sure this disclaimer should
> > > be deleted and replaced by just a standard GPL-2.0-or-later tag.
> >
> > I am inclined to agree with Steve. It's materially different from the
> > disclaimers in GPLv2 itself and could also indicate the remnants of an
> > otherwise vanished BSD license notice.
>
> The commit log [1] shows this was clearly always this way and is
> original code [2]
> People do funky stuff at times and mix and match notices and
> disclaimers from various origins.
> So this is IMHO a GPL alright and there is no BSD in play.

Hi Philippe, I am not sure it is necessarily so simple.

First, from an SPDX conformance perspective (I assume this whole
exercise aims to be SPDX conformant, although I'm not really clear on
that), it is not legitimately describable as merely "GPL-2.0-or-later"
because the disclaimer text is text not found in, and which does not
match the SPDX license template for, GPLv2. To argue otherwise is to
assert that any existing SPDX identifier is validly applied even where
arbitrary warranty disclaimer or liability-limitation language is
added to a license document or llicense notice -- the SPDX
specification should be amended if this is the desired policy, but
that would seem to require making some potentially significant legal
judgments which I had thought SPDX was designed to avoid.

Second, from a legal (and open source licensing folk-understanding)
perspective, it is something other than merely "GPL-2.0-or-later",
because the added disclaimer has different legally-significant content
from otherwise corresponding language in GPLv2.

GPLv3 says: "for material you add to a covered work, you may (if
authorized by the copyright holders of that material) supplement the
terms of this License with terms:
a) Disclaiming warranty or limiting liability differently from the
terms of sections 15 and 16 of this License;" , which is treated as an
allowable additional restriction. And this codified an occasional
practice under GPLv2 which I believe the FSF had authorized. So if one
copyright holder decided to tack on the BSD disclaimer language to an
otherwise standard GPLv2 notice, that presumably should be treated
like an additional restriction, which needs to be left in place, and
perhaps even the appropriate thing is to define an SPDX identifier to
account for the additional language. (Given the way "exception" is
sort of defined at https://spdx.org/licenses/exceptions-index.html it
might not be appropriate to use the "WITH" syntax.)

Richard

  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
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 [this message]
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=CAC1cPGzUmbOrYZP9wth4fc639Kan+mXqrzB0DbNLofskVxYMJw@mail.gmail.com \
    --to=rfontana@redhat.com \
    --cc=linux-spdx@vger.kernel.org \
    --cc=pombredanne@nexb.com \
    --cc=swinslow@linuxfoundation.org \
    /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