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

Proposal to harmonize license for 2x helper classes #539

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bernie-laberge
Copy link
Contributor

@bernie-laberge bernie-laberge commented Aug 16, 2024

Harmonize license for 2x helper classes

Linked issues

NA

Summarize your change.

This commit is a proposal to harmonize the license for 2 helper classes.

Describe the reason for the change.

The previous license for these 2 helper classes is not an open-source license, so the project is unable to distribute these files with the project
This commit is a proposal to harmonize the license for 2 helper classes.

Context:
To comply with the Blackmagic SDK license, we had left the original Blackmagic license for the 2 utility classes that were adapted from the Blackmagic SDK example code.
Now since this is not an open-source license, we cannot legally add these utility classes to the Open RV ASWF repository.

Proposal:
To resolve this issue, we propose to remove the Blackmagic license from those 2 helper classes based on the rationale:

  • Those files define 2x C++ helper classes that merely wrap the Blackmagic SDK interface
  • Those files do not contain any Intellectual Property that needs to be protected.

Please let us know if you believe this is an acceptable solution or not.
Thanks

Describe what you have tested and on which operating system.

No code impact

Add a list of changes, and note any that might need special attention during the review.

If possible, provide screenshots.

@bernie-laberge bernie-laberge force-pushed the harmonize_license_for_utility_class branch from d91c301 to 7e29f71 Compare August 16, 2024 14:23
@bernie-laberge bernie-laberge changed the title Proposal to harmonize license for 2x utility classes Proposal to harmonize license for 2x helper classes Aug 16, 2024
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 this pull request may close these issues.

1 participant