All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Luis R. Rodriguez" <mcgrof@frijolero.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: rusty@rustcorp.com.au, linux-kernel@vger.kernel.org,
	Keith Packard <keithp@keithp.com>,
	Ralf Baechle <ralf@linux-mips.org>,
	David Woodhouse <dwmw2@infradead.org>,
	Stephen Hemminger <shemminger@vyatta.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: [PATCH] module: Clarify GPL-Compatible is OK
Date: Fri, 6 Apr 2012 17:51:51 -0700	[thread overview]
Message-ID: <CAB=NE6Xt3yeCvW-t7mbEWoYr04Rg1uvObAMTatAdpHtgsdZvYA@mail.gmail.com> (raw)
In-Reply-To: <CA+55aFy_TOEE-8RNmNKVz6xZ7s+f02jCVcDbA=MJY1iih=AJpg@mail.gmail.com>

On Fri, Apr 6, 2012 at 5:36 PM, Linus Torvalds
<torvalds@linux-foundation.org> wrote:
> On Fri, Apr 6, 2012 at 5:11 PM, Luis R. Rodriguez <mcgrof@frijolero.org> wrote:
>> -MODULE_LICENSE("Dual BSD/GPL");
>> +MODULE_LICENSE("GPL-Compatible");
>
> I really don't see the point.
>
> This makes things *worse*.
>
> "Dual BSD/GPL" actually tells you something: it tells you that you can
> take that code, and use it in a BSD project.
>
> In contrast "GPL-compatible" tells you nothing at all.
>
> So you are actually removing real information, and just making things
> harder for everybody.

Its a good point that we are not declaring the exact license used for
software, and while that is useful the "Dual BSD/GPL" tag is
misleading. As I see it there are four options:

1) Use this as a technical artifact only to ensure symbols we declare
EXPORT_SYMBOL_GPL() will only be used by GPL-Compatible modules. Also
use the GPL-Compatible tag as annotated by this patch to annotate
this. Then add another tag to specific the exact license, which is not
anything of an artifact but just informational to the binary module
but also software developer reviewing code. This last part would
clarify the exact license.

2) We keep extending the list of MODULE_LICENSE() with all the
different GPL-Compatible licenses we are comfortable with. This list
is pretty outdated already. This means we keep chugging along and
adding more licenses.

3) Leave things as is, and clarify this. I think this confuses
developers though, and for sharing purposes it would be nice. Hence
the patch. You have no idea how many e-mails I have had to deal with
to address this. People really think this is impossible. In fact we
had a flamewar eons ago because a few of us didn't know this was
possible to help the BSDs. Not just developers, I think there are even
maintainers not too sure about this.

4) Use the patch and leave it to the person who wants to extract code
to figure out the exact module license.

Option 1) seems to me to provide the best alternative but leaves open
then the question of whether or not we need to keep tabs of accepted
GPLv2 compatible licenses we accept or leave this as informational.
Option 4) handles the technical artifacts we care about but gives some
homework to consumers.

Please let me know if anyone can think of better alternatives.

  Luis

  reply	other threads:[~2012-04-07  0:52 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-07  0:11 [PATCH] module: Clarify GPL-Compatible is OK Luis R. Rodriguez
2012-04-07  0:27 ` Greg Kroah-Hartman
2012-04-07  0:28 ` Al Viro
2012-04-07  0:57   ` Luis R. Rodriguez
2012-04-07  0:36 ` Linus Torvalds
2012-04-07  0:51   ` Luis R. Rodriguez [this message]
2012-04-07  1:02     ` Luis R. Rodriguez
2012-04-08 12:42       ` Arend van Spriel
2012-04-07  2:49     ` Ted Ts'o
2012-04-07  3:01       ` Luis R. Rodriguez
2012-04-07 21:15         ` Ted Ts'o
2012-04-08  0:52           ` Luis R. Rodriguez
2012-04-08 14:57             ` Alan Cox
2012-04-08 16:06               ` Luis R. Rodriguez
2012-04-08 17:12                 ` Alan Cox
2012-04-08 20:23             ` Ted Ts'o
2012-04-07 19:03 ` Alan Cox
2012-04-08 12:49   ` Arend van Spriel
2012-04-08 22:50 ` Dan Williams
2012-05-07  2:39 ` Rusty Russell

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='CAB=NE6Xt3yeCvW-t7mbEWoYr04Rg1uvObAMTatAdpHtgsdZvYA@mail.gmail.com' \
    --to=mcgrof@frijolero.org \
    --cc=dwmw2@infradead.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=keithp@keithp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=ralf@linux-mips.org \
    --cc=rusty@rustcorp.com.au \
    --cc=shemminger@vyatta.com \
    --cc=torvalds@linux-foundation.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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.