All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Kevin Wolf <kwolf@redhat.com>
Cc: mdroth@linux.vnet.ibm.com, qemu-devel@nongnu.org,
	lcapitulino@redhat.com, Anthony Liguori <aliguori@amazon.com>,
	pbonzini@redhat.com, Wenchao Xia <xiawenc@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH RFC 2/3] qapi script: add support of event
Date: Mon, 13 Jan 2014 11:08:45 +0100	[thread overview]
Message-ID: <87ob3gb3de.fsf@blackfin.pond.sub.org> (raw)
In-Reply-To: <87sit0m1j2.fsf@blackfin.pond.sub.org> (Markus Armbruster's message of "Tue, 07 Jan 2014 13:14:57 +0100")

Ping^2!

Markus Armbruster <armbru@redhat.com> writes:

> Ping?
>
> Markus Armbruster <armbru@redhat.com> writes:
>
>> [Licensing problem, cc: Anthony]
>>
>> Kevin Wolf <kwolf@redhat.com> writes:
>>
>>> Am 13.12.2013 um 14:31 hat Eric Blake geschrieben:
>>>> On 11/12/2013 06:44 PM, Wenchao Xia wrote:
>>>> > +++ b/scripts/qapi-event.py
>>>> > @@ -0,0 +1,355 @@
>>>> > +#
>>>> > +# QAPI event generator
>>>> > +#
>>>> > +# Copyright IBM, Corp. 2013
>>>> > +#
>>>> > +# Authors:
>>>> > +#  Wenchao Xia <xiawenc@linux.vnet.ibm.com>
>>>> > +#
>>>> > +# This work is licensed under the terms of the GNU GPLv2.
>>>> 
>>>> Can you please use GPLv2+ (that is, add the "or later" clause)?  We
>>>> already have GPLv2-only code, but I don't want to increase the size of
>>>> that unfortunate license choice.
>>>
>>> In fact, it's even worse:
>>>
>>> +# This work is licensed under the terms of the GNU GPLv2.
>>> +# See the COPYING.LIB file in the top-level directory.
>>>
>>> These two lines contradict each other, COPYING.LIB contains the
>>> LGPL 2.1. The same bad license header is in the other QAPI generator
>>> scripts, so it's only copy&paste here.
>>
>> Specifically:
>>
>>     File                        Commit
>>     scripts/qapi-commands.py    c17d9908
>>     scripts/qapi-visit.py       fb3182ce
>>     scripts/qapi-types.py       06d64c62
>>     scripts/qapi.py             0f923be2
>>
>> All four from Michael Roth via Luiz.
>>
>>> This doesn't make things easier, because if things are copied, the
>>> license of the source must be respected. And it seems rather dubious to
>>> me what this license actually is. If it's GPLv2-only, we can't just
>>> change it in the new copy.
>>
>> IANAL, and I wouldn't dare to judge which of the two conflicting license
>> claims takes precedence.  Possibly neither, and then the files might
>> technically not be distributable.
>>
>> Anyway, this mess needs to be addressed.  Michael, what was your
>> *intended* license?
>>
>> If it wasn't GPLv2+, then why?
>>
>> Do we need formal ACKs from all contributors to fix the licensing
>> comment in these four files?

  reply	other threads:[~2014-01-13 10:09 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-13  1:44 [Qemu-devel] [PATCH RFC 0/3] add direct support of event in qapi schema Wenchao Xia
2013-11-13  1:44 ` [Qemu-devel] [PATCH RFC 1/3] os-posix: include sys/time.h Wenchao Xia
2013-12-13 13:26   ` Eric Blake
2013-11-13  1:44 ` [Qemu-devel] [PATCH RFC 2/3] qapi script: add support of event Wenchao Xia
2013-11-28  0:48   ` Luiz Capitulino
2013-11-28  7:16     ` Wenchao Xia
2013-11-28 14:31       ` Luiz Capitulino
2013-12-02  6:48         ` Wenchao Xia
2013-12-13  3:00           ` Wenchao Xia
2013-12-13 13:31   ` Eric Blake
2013-12-13 13:43     ` Kevin Wolf
2013-12-16  2:50       ` Wenchao Xia
2013-12-16  9:04         ` Markus Armbruster
2013-12-16  9:13       ` Markus Armbruster
2014-01-07 12:14         ` Markus Armbruster
2014-01-13 10:08           ` Markus Armbruster [this message]
2014-01-15  1:47             ` Wenchao Xia
2014-01-16  2:50         ` Michael Roth
2014-01-16 11:05           ` Paolo Bonzini
2014-01-30 10:35             ` Markus Armbruster
2013-12-16  2:51     ` Wenchao Xia
2013-11-13  1:44 ` [Qemu-devel] [PATCH 2/2] try Wenchao Xia
2013-11-13  1:47   ` Wenchao Xia
2013-11-13  1:44 ` [Qemu-devel] [PATCH RFC 3/3] tests: add test cases for qapi event support Wenchao Xia
2013-12-13 13:36   ` Eric Blake
2013-12-16  2:55     ` Wenchao Xia
2013-12-16  3:12       ` Wenchao Xia
2014-01-02  2:30         ` Wenchao Xia
2013-11-22  2:03 ` [Qemu-devel] [PATCH RFC 0/3] add direct support of event in qapi schema Wenchao Xia
2013-11-22 14:33   ` Luiz Capitulino

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=87ob3gb3de.fsf@blackfin.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=aliguori@amazon.com \
    --cc=kwolf@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=xiawenc@linux.vnet.ibm.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
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.