Re: [Libguestfs] [PATCH libnbd v2 1/2] golang: Changes test license to LPGL

On Mon, Nov 01, 2021 at 11:30:07AM +0000, Richard W.M. Jones wrote: > On Mon, Nov 01, 2021 at 09:32:37AM +0000, Daniel P. Berrangé wrote: > > On Sun, Oct 31, 2021 at 06:20:05PM +0000, Richard W.M. Jones wrote: > > > On Sun, Oct 31, 2021 at 06:59:32PM +0200, Nir Soffer wrote: > > > > Having different license for the tests complicates everyone life for no > > > > benefit. Change the license to LGPL2+ like the rest of the library. > > > > > > > > Related discussion: > > > > https://listman.redhat.com/archives/libguestfs/2021-October/msg00196.html > > > > > > > > Signed-off-by: Nir Soffer <nsoffer(a)redhat.com&gt; > > > > > > I agree with this change. > > > > > > Since this involves some code that I wrote originally, can you add: > > > > > > Signed-off-by: Richard W.M. Jones <rjones(a)redhat.com&gt; > > > > > > However I'm not the only author here (by quite a lot!): > > > > > > $ git shortlog -s golang/src/libguestfs.org/libnbd/ > > > 13 Eric Blake > > > 2 Nir Soffer > > > 4 Richard W.M. Jones > > > > > > so could be a good idea to ask Eric specifically for permission too > > > (even though in both cases the © is likely assigned to Red Hat as our > > > employer). > > > > Looking at libnbd.git as a whole I see > > > > 1 Author: anson <83398016+anson-lo(a)users.noreply.github.com&gt; > > 1 Author: Chris Lamb <lamby(a)debian.org&gt; > > 324 Author: Eric Blake <eblake(a)redhat.com&gt; > > 1 Author: Laszlo Ersek <lersek(a)redhat.com&gt; > > 52 Author: Martin Kletzander <mkletzan(a)redhat.com&gt; > > 5 Author: Nir Soffer <nirsof(a)gmail.com&gt; > > 33 Author: Nir Soffer <nsoffer(a)redhat.com&gt; > > 2 Author: Pino Toscano <ptoscano(a)redhat.com&gt; > > 936 Author: Richard W.M. Jones <rjones(a)redhat.com&gt; > > > > > > only two patches from non-Red Hat people, and both of those patches > > are trivial bug fixes so could be said to be non-copyrigtable changes. > > With this in mind, I would suggest that it is viable to remove all the > > license headers across the codebase and add SPDX tags in their place. > > While viable, we did discuss this already and decided against it for > now -- we'll watch what qemu & libvirt are doing. QEMU / libvirt aren't liekly to adopt SPDX for their main repos because they both have a huge variety of copyright holders, so it is not viable to get permission to replace the license headers with SPDX tags. The kernel adopted SPDX despite not getting permission from all copyright holders, but they went through some kind of auditing process with legal oversight to have confidence that is is OK. I don't know enough about the fine details to be able to suggest copying that approach. libnbd is fortunately not in this situation because it is essentially all (C) Red Hat. IOW, don't wait for libvirt/QEMU - decide it for libnbd on its own merits. FWIW, I'm using SPDX tags for any new projects I create from scratch going forward e.g. https://gitlab.com/bichon-project/bichon/