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

Add full license info for voxel textures #1113

Merged
merged 2 commits into from
Sep 15, 2024

Conversation

Montandalar
Copy link
Contributor

Add full license info for voxel textures

Hello,

The textures in the Voxel Demo were included from Minetest Game ("MTG") for Minetest without "full" attribution of the authors. The attribution information needs to be present since the files are under CC BY-SA 3.0.

Although a general attribution was given to the "Minetest Contributors", I thought it was worth noting the original authors of each file that was used, and noting that some are actually under CC0. This info was supplied in a README.

Although CC0 is compatible with CC BY-SA 3.0 and the whole work can be received under CC BY-SA 3.0, it would be good for people to know there are parts they can reuse with less restriction. I think the current wording and linking to "MTG" makes it clear enough if someone wants to attempt such reuse.

@aaronfranke
Copy link
Member

I don't think it's legally necessary to list the CC0 textures, but maybe it's a good idea to do it anyway. I'll wait a bit in case other contributors want to give feedback on this PR.

On a technical level, I suggest using ` characters around file names instead of using \_.

3d/voxel/README.md Outdated Show resolved Hide resolved
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Calinou Calinou merged commit fdae025 into godotengine:master Sep 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants