Re: [PATCH rdma-core 3/4] debian/copyright: Add copyright information

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 29, 2016 at 03:49:07PM +0200, Leon Romanovsky wrote:
> From: Talat Batheesh <talatb@xxxxxxxxxxxx>
> 
> This patch adds debian/copyright file and list all
> different licences and copyrights, according to
> debian policy.

> +Files: *
> +License: GPL-2+ or BSD

We don't have anything that includes the 'any later version' clause in
the copyright header required to be GPLv2+ - so everything should be simply
'GPL-2'

> +Files: debian/*
> +Copyright: 2016 Talat Batheesh <talatb@xxxxxxxxxxxx>
> +License: GPL-2+

You can add me here too

> +Files: buildlib/cbuild
> +Copyright: 2015, 2016, Obsidian Research Corp. See COPYING.
> +License: GPL-2+ or BSD
> +
> +Files: buildlib/fixup-include/*
> +Copyright: 2016, Mellanox Technologies Ltd.
> +License: GPL-2+ or BSD

Many of these copyright attributions are a bit wonky :\ I personally
think this format of Debian's is too difficult.

> +Files: ccan/*
> +Copyright: unspecified
> +License: CC0 and BSD-MIT
> +
> +Files: ibacm/*
> +Copyright: 2013-2015, Mellanox Technologies LTD.
> +  2009-2014, Intel Corporation.
> +License:  GPL-2 or OpenIB.org BSD (FreeBSD variant)

There is no text stanza for 'OpenIB.org BSD'

> +Files: providers/ipathverbs/*
> +Copyright: 2006-2009, QLogic Corp.
> +  2005
> +License: GPL-2+ or BSD

ipath has a special license. Please review COPYING.md and make sure
all the special cases called out are covered properly.

> +License: GPL-2+ or BSD

I think this should just be 'BSD', same with all the 'or' stanzas.

I didn't see the intel bsd license in here (see
providers/hfi1verbs/hfiverbs.c, it is slightly different again)

It would be good to update COPYING.md to refer people to this new file
as it is better than the short summary.

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux