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

Yarn2nix-moretea is GPL-3.0 licensed, should be MIT #334267

Closed
adisbladis opened this issue Aug 13, 2024 · 9 comments · Fixed by #337128
Closed

Yarn2nix-moretea is GPL-3.0 licensed, should be MIT #334267

adisbladis opened this issue Aug 13, 2024 · 9 comments · Fixed by #337128
Labels
0.kind: bug Something is broken 1.severity: legal This PR or issue raises or fixes a legal issue, e.g. licensing compliance

Comments

@adisbladis
Copy link
Member

adisbladis commented Aug 13, 2024

Describe the bug

Problem

All expressions in nixpkgs are supposed to be MIT licensed, but yarn2nix-moretea is licensed under GPL-3.0.

yarn2nix-moretea as an externally developed project under different licensing terms that we use for nixpkgs.
It was merged into nixpgks in #108138 without the license being changed to MIT or a compatible license.

Impact

Every package using mkYarnPackage is using improperly licensed expressions, which at the time of writing looks to be a bit more than 25 packages.

Solution

I have two proposed solutions, neither of which are particularly appealing:

  1. Re-license yarn2nix-moretea under MIT

This would require assembling relicensing signatures from all existing yarn2nix authors which include both contributors when it was in nix-community & in nixpkgs.
It's gonna take some time and keep us in license non-compliance in the mean time.

  1. Drop yarn2nix, migrate packages to something else

The newly added yarnBuildHook looks like a good contender as pointed out by @SuperSandro2000 in chat.

CC

CCing authors and other relevant parties


Add a 👍 reaction to issues you find important.

@adisbladis adisbladis added the 0.kind: bug Something is broken label Aug 13, 2024
@emilazy
Copy link
Member

emilazy commented Aug 13, 2024

Just to repeat what I said on Matrix, if we can’t figure out something better in a few days I will open a relicensing issue and start the work of pinging all the contributors. Reimplementing it sounds nice, but I suspect it would take quite a while given the speed at which Nixpkgs language ecosystems tend to move.

@SuperSandro2000
Copy link
Member

yarn2nix can be easily misused and trigger IFD. I think too that in the long run we should replace it.

Maybe yarnBuildHook/yarnConfigHook from #318015 are already enough and the package can easily be migrated?

@SuperSandro2000 SuperSandro2000 added 0.kind: enhancement Add something new 1.severity: legal This PR or issue raises or fixes a legal issue, e.g. licensing compliance and removed 0.kind: bug Something is broken labels Aug 13, 2024
@emilazy emilazy added 0.kind: bug Something is broken and removed 0.kind: enhancement Add something new labels Aug 13, 2024
@emilazy
Copy link
Member

emilazy commented Aug 13, 2024

(Nixpkgs potentially actually being GPLv3 is very much a bug IMO.)

Migration is all well and good, but we probably ought to try to relicense anyway: we can’t change old revisions of Nixpkgs and it’s not good for multiple years of Nixpkgs history to be under unclear licensing conditions.

@Lassulus
Copy link
Member

I guess re-licensing is the better way for now? We can ping all the contributors, from a glance I read most of the names before.

@emilazy
Copy link
Member

emilazy commented Aug 13, 2024

Yes, I think we should probably at least try. I will open a new GitHub issue for it today so that we can discuss here without it getting bogged down in rote responses.

@emilazy
Copy link
Member

emilazy commented Aug 13, 2024

Okay, here we go I guess: #334374

@WilliButz
Copy link
Member

WilliButz commented Aug 13, 2024

Oof, thank you @emilazy for trying to clean up that mess.
I just got the notification from the tracking issue and I feel that a big portion of any blame should go towards my direction because of the introduction in #60429, rather than #108138
While I (try to) put a lot more thought into licenses today, it didn't cross my mind back then that vendoring with the included license could have such implications.

@emilazy
Copy link
Member

emilazy commented Aug 13, 2024

No worries @WilliButz – thanks for trying to improve the Node packaging situation in Nixpkgs and for your quick reply to the ping! This is an institutional failure IMO, and I think that it would have been caught before merge these days.

@dotlambda
Copy link
Member

dotlambda commented Aug 13, 2024

@petabyteboy (GitHub account is gone; will try to follow up via email unless anyone knows other current contact information)

https://discourse.nixos.org/u/petabyteboy seems still active. It might be worth posting the same as #334374 on Discourse if you want people's replies to be public (unlike with email).

github-actions bot pushed a commit that referenced this issue Aug 27, 2024
All contributors that have made contributions at all
likely to be copyrightable have agreed to relicense this
code under the same MIT licence as the rest of Nixpkgs in
<#334374>.

Note that this applies to all previous versions of the code, including
in older Nixpkgs versions where the GPLv3 text is still present and the
original yarn2nix repositories before they were imported into Nixpkgs.

Closes: #334267
Closes: #334374
(cherry picked from commit df1aa35)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug Something is broken 1.severity: legal This PR or issue raises or fixes a legal issue, e.g. licensing compliance
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants