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

no license #34

Closed
SArpnt opened this issue Apr 4, 2023 · 3 comments · Fixed by #35
Closed

no license #34

SArpnt opened this issue Apr 4, 2023 · 3 comments · Fixed by #35

Comments

@SArpnt
Copy link

SArpnt commented Apr 4, 2023

(i am not a lawyer)

as far as i can tell, this project has no license. i don't think i or other people can legally use or contribute to it without a license.

also, the readme says it's based on cargo-psp, which is quite vague. if it uses its code, i'm pretty sure it needs the mit license of cargo-psp somewhere in this repo, and the PSPSDK license (since cargo-psp uses it)

@Techie-Pi
Copy link
Member

Techie-Pi commented Apr 4, 2023

I think that after #26 the code diverged significantly from cargo-psp, so we may be able to use whatever license we want due to most code being ours (though I'm not a lawyer nor have I worked in this repo 😄)

@SArpnt
Copy link
Author

SArpnt commented Apr 4, 2023

you'd be able to use whatever license you want anyways since mit isn't share alike. it would just require the notice to be copied over.

either way i think the best option is to give this repo an mit license and include the one from cargo-psp since it's safer and convenient for other projects. gpl, agpl, and various other licenses would also work but that's probably just going to confuse people that don't quite understand how the licenses work, and it's extremely unlikely anyone would make a closed source fork

@Techie-Pi
Copy link
Member

We mostly use MIT/Apache 2.0 and Zlib, so we'll probably stick to that.
@rust3ds/active Any thoughts?

@Meziu Meziu mentioned this issue Apr 5, 2023
@Meziu Meziu closed this as completed in #35 Apr 5, 2023
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.

2 participants