-
Notifications
You must be signed in to change notification settings - Fork 33
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 "license-org" and "license-text" parameters #298
base: master
Are you sure you want to change the base?
Conversation
@allenh1 Could you please review and comment? |
If "license-org" and license-text" are passing in command line, the default value will be overlayed in generated files. Signed-off-by: Jason Shi <[email protected]>
@JasonShigit sorry for the delay! I'm running CI on it now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with green CI
Signed-off-by: Jason Shi <[email protected]>
Hi @allenh1 |
@JasonShigit it seems there are still a few errors to resolve. The best way to run locally is to install all the requirements (see |
Thanks for submitting this. Can you describe the use case for allowing superflore to generate recipes under a difference license? Since the recipes are generated based on metadata, I am skeptical whether they can be copyrighted in the first place. Moreover the default copyright used for bitbake recipes is MIT and this patch would change the default to BSD. If you can describe the reason for this change it will help me better understand why it is needed. I think before I could merge this in, I would need to run it by the OSRA. |
If "license-org" and license-text" are passing in command line, the default value will be overlayed in generated files.