Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

License incompatability: mixes GPL 2 only plus GPL 3 codes #23

Closed
mr-c opened this issue Apr 4, 2020 · 9 comments
Closed

License incompatability: mixes GPL 2 only plus GPL 3 codes #23

mr-c opened this issue Apr 4, 2020 · 9 comments

Comments

@mr-c
Copy link

mr-c commented Apr 4, 2020

While preparing this module for Debian, it was noticed that there is an incompatible mixture of GPL-2 and GPL-3 code:

Files: R/as.edgelist.R
Copyright: 2003-2015 Statnet Commons
License: GPL-3

Files: inst/network.api/networkapi.c
 inst/network.api/networkapi.h
 src/Rinit.c
 src/access.c
 src/access.h
 src/constructors.c
 src/constructors.h
 src/layout.c
 src/layout.h
 src/utils.c
 src/utils.h
Copyright: Carter T. Butts <buttsc@uci.edu>
           Jeffrey Horner <jeffrey.horner@gmail.com>
License: GPL-2

As per https://www.gnu.org/licenses/gpl-faq.en.html#AllCompatibility it is not permissible to mix GPL-2 and GPL-3 code.

Please consider working with the copyright holders to re-license into a compatible combination. Perhaps everything under GPL-3 or GPL-3+ would be simplest.

Thanks!

@mr-c
Copy link
Author

mr-c commented Apr 23, 2020

To be clear, until this incomparability is resolved, Debian can not package and release this module or any module that depends on it

@mbojan
Copy link
Member

mbojan commented Apr 24, 2020

Thanks! Calling upon @CarterButts @martinamorris @krivit

@mbojan
Copy link
Member

mbojan commented Apr 25, 2020

d6a2a50

@mr-c
Copy link
Author

mr-c commented May 4, 2020

Thanks!

There are two more files that still need updating:

GPL 2.0 only:
https://github.com/statnet/network/blob/master/inst/network.api/networkapi.h and .c

@krivit
Copy link
Member

krivit commented Sep 21, 2020

@mr-c, addressed in a7575e2 , by the looks of it.

@krivit krivit closed this as completed Sep 21, 2020
@mr-c
Copy link
Author

mr-c commented Oct 9, 2020

@krivit Can you make a new release so that the relicensing is clear?

@mr-c
Copy link
Author

mr-c commented Oct 9, 2020

(Many thanks, by the way!)

@krivit
Copy link
Member

krivit commented Oct 11, 2020

@mr-c , fairly soon, I think. @CarterButts ?

@CarterButts
Copy link
Contributor

Argh, that's my mistake - I just pushed 1.16.1 and missed those two files. They are now fixed in that branch on GH, so if another bugfix release is pushed to CRAN from the 1.16.* line it will get the fixes. They are also fixed in the 1.17 line, so the next release will contain all of those changes. Sorry about that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants