All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Radim Krčmář" <rkrcmar@redhat.com>
To: Thomas Huth <thuth@redhat.com>
Cc: pbonzini@redhat.com, kvm@vger.kernel.org
Subject: Re: [kvm-unit-tests PATCH] lib: Add headers to generic library files
Date: Tue, 25 Apr 2017 17:32:28 +0200	[thread overview]
Message-ID: <20170425153228.GH5713@potion> (raw)
In-Reply-To: <50161dcd-c619-22cd-5707-745744e52c92@redhat.com>

2017-04-25 09:11+0200, Thomas Huth:
> On 07.04.2017 15:49, Andrew Jones wrote:
>> On Fri, Apr 07, 2017 at 03:16:57PM +0200, Thomas Huth wrote:
>>> Many files in the lib folder do not have proper statements about
>>> their license. Add such a header there so that it is clear under
>>> which conditions the code can be used.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>> diff --git a/lib/argv.c b/lib/argv.c
>>> @@ -1,3 +1,10 @@
>>> +/*
>>> + * Set up arguments for main() and prepare environment variables
>>> + *
>>> + * This code is free software; you can redistribute it and/or modify it
>>> + * under the terms of the GNU Library General Public License version 2.
>>> + */
>>> +
>>>  #include "libcflat.h"
>>>  #include "auxinfo.h"
>>>  
>> 
>> Reviewed-by: Andrew Jones <drjones@redhat.com>
> 
> *ping*

(Sorry, I'll need to double-check all unit test patches from that time.)

> Paolo, Radim, could you pick this up please if it sounds OK for you?

It does, the licensing follows COPYRIGHT and I assume that we can't
change the license without a lot of legal hassle.

Took me a while to realize what isn't clear: libcflat.h is GPL 2 and
being explicit with the border between GPL 2 and LGPL 2 sounds good for
re-usability of the code.  (I assume that the whole kvm-unit-tests is
effectively GPL 2, but parts of it are more benevolent.)

Applied, thanks.

  reply	other threads:[~2017-04-25 15:32 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-07 13:16 [kvm-unit-tests PATCH] lib: Add headers to generic library files Thomas Huth
2017-04-07 13:49 ` Andrew Jones
2017-04-25  7:11   ` Thomas Huth
2017-04-25 15:32     ` Radim Krčmář [this message]
2017-04-26  9:47       ` Thomas Huth

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=20170425153228.GH5713@potion \
    --to=rkrcmar@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --cc=thuth@redhat.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.