-
Notifications
You must be signed in to change notification settings - Fork 24
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
Added clarification to README #526
Conversation
Before we merge this, I'll email the FSF regarding the topic of JSON models, since (unless I'm mistaken and the JSON models aren't from the vanilla resource pack) they could actually count as code under GPL (and thus restricting them would violate GPL) EDIT: I've sent them the email. Unlike Mojang, the FSF is more concerned with making sure open source software stays compliant with GPL, so I'm not too concerned with them asking for MCprep to be taken down. Here's a copy of the email I sent them:
|
Yeah, looks like whatever we do, it's going to be a grey area:
The key part is this:
I think for now, we'll just add the clarification, and then work towards not distributing Minecraft assets (instead asking users to provide a copy of the Minecraft JAR file for assets). I have sent a follow-up regarding requiring users to supply the Minecraft assets themselves, but I think it's fine if we merge this for now. |
Ahh, just saw the page update now, will approve and merge then. In terms of incremental steps we could do, the main one is the distribution of the overall default resource pack. I already use an entirely automated script to populate these files and mappings, so it's not unreasonable to have it run on end-user machines. Although, for reasons like #529, it can lead to other new issues we can't prevent ahead of time. |
Added clarification that the new rule only applies to assets (applying to all of MCprep is a violation of GPL as I later realized)