-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Verify that all asset licenses are accounted for in CREDITS.md
#5399
Comments
For anyone who's willing to work on this: Information about many of the assets can be found here: Lines 17 to 27 in 619c30c
However, it's very incomplete:
And some words of advice: I'd highly recommend using SPDX License Identifiers to refer to the licenses in the CSV. Your CI pipeline should confirm that all the licenses are compatible with Bevy's MIT/Apache 2.0 license structure. And once you figure out this mess, |
When creating that sound I assumed it would just fall under the bevy license, I didn't give more thoughts than this, but if you need me to agree to any other license I will gladly do what it takes. |
IMO we should remove CREDITS.md once this is done; we really don't want two sources of truth. |
Perhaps we could generate CREDITS.md from the CSV using an action (similar to the README for the examples). But that would make things more complicated. Deleting it is probably the best solution. |
I generated that particular image by stacking 4 textures from https://freepbr.com/ I can try to find the original links. |
Thanks! While we do have the usage rights for those 4 textures, the licensing terms are quite vague. Perhaps we could get in touch with Brian (the owner of Free PBR) and ask him to relicense them under something like the Creative Commons Attribution-NonCommercial 4.0 International license, which is pretty similar to what he currently has. |
What problem does this solve or what need does it fill?
As discussed in #4975 (comment), we should verify that all assets are correctly attributed.
What solution would you like?
LICENSES.csv
file to theassets
folder.What alternative(s) have you considered?
Manually verify that these are correct.
This solves the problem for now, but doesn't increase our long-term confidence.
The text was updated successfully, but these errors were encountered: