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

'terminate' package uses GPL-2.0 license where 'og-image' uses MIT #212

Closed
sethidden opened this issue May 22, 2024 · 3 comments · Fixed by #217
Closed

'terminate' package uses GPL-2.0 license where 'og-image' uses MIT #212

sethidden opened this issue May 22, 2024 · 3 comments · Fixed by #217

Comments

@sethidden
Copy link
Contributor

sethidden commented May 22, 2024

It's used in the package runtime (not as a development tool) here

installChromeProcess.pid && terminate(installChromeProcess.pid)

A change introduced in this commit: e0cdde7

License SPDX in terminate: https://github.com/dwyl/terminate/blob/main/package.json#L32
Also see:

As per GPL-2.0 section 2

  1. You may modify your copy or copies of the Program or any portion of it, thus forming a work based on the Program, and copy and distribute such modifications or work under the terms of Section 1 above, provided that you also meet all of these conditions:
    ...
    b) You must cause any work that you distribute or publish, that in whole or in part contains or is derived from the Program or any part thereof, to be licensed as a whole at no charge to all third parties under the terms of this License.

Shouldn't this package be GPL-ed or stop using terminate to keep using MIT?

@manniL
Copy link

manniL commented May 29, 2024

cc @harlan-zw

@sethidden
Copy link
Contributor Author

sethidden commented Jun 3, 2024

@harlan-zw I read your post in terminate. Could you let me know the particular doubts you're having in regards to the GPL/MIT license?

@harlan-zw
Copy link
Collaborator

harlan-zw commented Jun 5, 2024

I was looking for the maintainer to comment ideally.

For context, we need this package as occasionally browser gets stuck open in the CI.

I don't have capacity right now to replace this package with bespoke code or find an alternative fix so changing the license upstream would be ideal. PRs are also welcome of course :)

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

Successfully merging a pull request may close this issue.

3 participants