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

incorrect GPL headers #1890

Closed
cgwalters opened this issue Aug 23, 2019 · 3 comments
Closed

incorrect GPL headers #1890

cgwalters opened this issue Aug 23, 2019 · 3 comments
Assignees

Comments

@cgwalters
Copy link
Member

I just noticed that a lot of code has a GPLv2+ header, not LGPLv2+ which is what our COPYING file says.

(Also related: #1179 )

Running git log -G 'under the terms of the GNU General Public' and looking at the log in reverse, f93cb5b is the first commit that imported a GPLv2+ test driver.

However, the next hit after that is c0d15a6 which added some daemon code as GPLv2+, and I don't think anyone noticed at the time...

And since when creating new C files at least I often copy/paste a previous one as a basis, I think it spread.

The first introduction of Rust code in 479406e was GPLv2+ too...which...that code originated in coreos/coreos-assembler@35bd3c6 and didn't have a license header at the time (oops). I guess Jonathan added that, would be curious if he explicitly chose GPLv2+ for a reason.

So um...I am not sure we can easily dig ourselves out of this hole now. We may have to just accept reality and dual-license LGPLv2+/GPLv2+.

However, I would like to take the stance that new Rust code we write should be the "standard Rust" dual MIT/ASL2.0, and new glib-C code should be LGPLv2+. Opinions?

@cgwalters
Copy link
Member Author

And also, we clearly need a license CI test...

@jlebon
Copy link
Member

jlebon commented Sep 5, 2019

The first introduction of Rust code in 479406e was GPLv2+ too...which...that code originated in coreos/coreos-assembler@35bd3c6 and didn't have a license header at the time (oops). I guess Jonathan added that, would be curious if he explicitly chose GPLv2+ for a reason.

Ouch... I did not. It's very likely I just copied the header from one of the C sources, which happened to be one of the daemon files which used GPLv2+. 😢

However, I would like to take the stance that new Rust code we write should be the "standard Rust" dual MIT/ASL2.0, and new glib-C code should be LGPLv2+. Opinions?

SGTM! Might be too late for src/daemon/ but at least rust/ is new enough that it should just be a handful of people we'd need to ask if we want to relicense the existing code.

Checking now:

$ git shortlog -s --no-merges rust/
    73  Colin Walters
    39  Jonathan Lebon
     3  Rafael Fonseca

So that just leaves @r4f4

@r4f4
Copy link
Contributor

r4f4 commented Sep 5, 2019

No problems by me.

@cgwalters cgwalters self-assigned this Sep 5, 2019
cgwalters added a commit to cgwalters/rpm-ostree that referenced this issue Sep 5, 2019
We have contacted all contributors to the code in `rust/` and
that code is now all relicensed under the "standard Rust license"
of `Apache 2.0 OR MIT`.

[Due to an accident](coreos#1890),
some GPLv2+ code was imported in the C side, and we're unlikely to
easily change that now.  Make this more official by adding the GPLv2.

I'd like to go through the C code and add SPDX and possibly investigate
relicensing some of the GPLv2+ code to LGPLv2+ but, not right now.

For a bit more about Rust and SPDX, see [this issue](rust-lang/cargo#2039).
cgwalters added a commit to cgwalters/rpm-ostree that referenced this issue Sep 5, 2019
We have contacted all contributors to the code in `rust/` and
that code is now all relicensed under the "standard Rust license"
of `Apache 2.0 OR MIT`.

[Due to an accident](coreos#1890),
some GPLv2+ code was imported in the C side, and we're unlikely to
easily change that now.  Make this more official by adding the GPLv2.

I'd like to go through the C code and add SPDX and possibly investigate
relicensing some of the GPLv2+ code to LGPLv2+ but, not right now.

For a bit more about Rust and SPDX, see [this issue](rust-lang/cargo#2039).

Closes: coreos#1890
cgwalters added a commit to cgwalters/rpm-ostree that referenced this issue Sep 5, 2019
We have contacted all contributors to the code in `rust/` and
that code is now all relicensed under the "standard Rust license"
of `Apache 2.0 OR MIT`.

[Due to an accident](coreos#1890),
some GPLv2+ code was imported in the C side, and we're unlikely to
easily change that now.  Make this more official by adding the GPLv2.

I'd like to go through the C code and add SPDX and possibly investigate
relicensing some of the GPLv2+ code to LGPLv2+ but, not right now.

For a bit more about Rust and SPDX, see [this issue](rust-lang/cargo#2039).

Closes: coreos#1890
cgwalters added a commit to cgwalters/rpm-ostree that referenced this issue Sep 5, 2019
We have contacted all contributors to the code in `rust/` and
that code is now all relicensed under the "standard Rust license"
of `Apache 2.0 OR MIT`.

[Due to an accident](coreos#1890),
some GPLv2+ code was imported in the C side, and we're unlikely to
easily change that now.  Make this more official by adding the GPLv2.

I'd like to go through the C code and add SPDX and possibly investigate
relicensing some of the GPLv2+ code to LGPLv2+ but, not right now.

For a bit more about Rust and SPDX, see [this issue](rust-lang/cargo#2039).

Closes: coreos#1890
rh-atomic-bot pushed a commit that referenced this issue Sep 5, 2019
We have contacted all contributors to the code in `rust/` and
that code is now all relicensed under the "standard Rust license"
of `Apache 2.0 OR MIT`.

[Due to an accident](#1890),
some GPLv2+ code was imported in the C side, and we're unlikely to
easily change that now.  Make this more official by adding the GPLv2.

I'd like to go through the C code and add SPDX and possibly investigate
relicensing some of the GPLv2+ code to LGPLv2+ but, not right now.

For a bit more about Rust and SPDX, see [this issue](rust-lang/cargo#2039).

Closes: #1890

Closes: #1897
Approved by: jlebon
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

3 participants