Re: [Xen-devel] [PATCH v2 hopefully for 4.7] libxenvchan: Change license of header from Lesser GPL v2.1 to BSD

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Mon, Jun 13, 2016 at 05:28:57AM -0400, Konrad Rzeszutek Wilk wrote:
> As the xen/COPYING file says:
> "A few files are licensed under both GPL and a weaker BSD-style
> license. This includes all files within the subdirectory
> include/public, as described in include/public/COPYING. All such files
> include the non-GPL license text as a source-code comment. Although
> the license text refers generically to "the software", the non-GPL
> license applies *only* to those source files that explicitly include
> the non-GPL license text."
> 
> The libxenvchan.h is under xen/include/public/io directory
> and the xen/include/public/COPYING says:
> 
> "XEN NOTICE
> ==========
> 
> This copyright applies to all files within this subdirectory and its
> subdirectories:
>   include/public/*.h
>   include/public/hvm/*.h
>   include/public/io/*.h
> 
> The intention is that these files can be freely copied into the source
> tree of an operating system when porting that OS to run on Xen. Doing
> so does *not* cause the OS to become subject to the terms of the GPL.
> 
> All other files in the Xen source distribution are covered by version
> 2 of the GNU General Public License except where explicitly stated
> otherwise within individual source files.
> "
> Having the libxenvchan.h as Lesser GPL v2.1 where the COPYING file
> says otherwise is confusing to say at least.
> 
> Upon consulting with the authors of libxenvchan they said:
> "FWIW Neither I, nor ITL staff (as author of original libvchan library)
> have anything against converting it to the BSD-style licence."
> (Marek Marczykowski-Górecki,
> http://lists.xen.org/archives/html/xen-devel/2016-06/msg00995.html)
> so as such lets change it.
> 
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Acked-by: Anil Madhavapeddy <anil@xxxxxxxxxx>
> Acked-by: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>
> Acked-by: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
> Acked-by: Jan Beulich <JBeulich@xxxxxxxx>
> Acked-by: Jason Andryuk <andryuk@xxxxxxxx>
> Acked-by: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
> Acked-by: Matthew Daley <mattjd@xxxxxxxxx>
> Acked-by: Olaf Hering <olaf@xxxxxxxxx>
> Acked-by: Roger Pau Monne <roger.pau@xxxxxxxxxxxxx>
> Acked-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> ["I have spoken to my line manager.  I can confirm that Citrix is happy
>  with this proposed change.  So:
> 
>  Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> 
>  This view from Citrix covers all contributions made to these files in
>  the course of Citrix's employees' employment, which I think is:
> 
>  > Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>  > cc: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
>  > Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>
>  > Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
>  > Cc: Roger Pau Monne <roger.pau@xxxxxxxxxxxxx>
>  > Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>
>  > Cc: Tim Deegan <tim@xxxxxxx>
>  > Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
> 
>  [in subsequent email]:
>  Wei points out that this ought also to include Keir Fraser's
>  contribution, which was (only) in 2012.
>  " (from Ian's email):
> 

The list of people who touched tools/libvchan:

    $ git log --format="%an <%ae>" tools/libvchan | sort | uniq
    Anil Madhavapeddy <anil@xxxxxxxxxx>
    Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>
    David Scott <dave.scott@xxxxxxxxxx>
    Ian Campbell <ian.campbell@xxxxxxxxxx>
    Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
    Jan Beulich <JBeulich@xxxxxxxx>
    Jason Andryuk <andryuk@xxxxxxxx>
    Keir Fraser <keir@xxxxxxx>
    Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
    Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
    Matthew Daley <mattjd@xxxxxxxxx>
    Olaf Hering <olaf@xxxxxxxxx>
    Roger Pau Monne <roger.pau@xxxxxxxxxxxxx>

Note that David Scott is listed in my output, but Konrad didn't have
him in the list of authors.

However, David Scott made his contribution while he was employed by
Citrix, so his contribution is covered by the statement provided by Ian.

(CC Dave)

I will update the commit message accordingly.

The list of people who touched public header:

   $ git log --format=format:"%an <%ae>" xen/include/public/io/libxenvchan.h | 
sort | uniq
   Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>
   Ian Campbell <ian.campbell@xxxxxxxxxx> 

All in all, I believe this patch has collected all the necessary acks.

I've queued it up for unstable and 4.7 branch.

Thanks everyone.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel