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

LGPL license on a source file #305

Closed
hyandell opened this issue Dec 10, 2023 · 2 comments · Fixed by #306
Closed

LGPL license on a source file #305

hyandell opened this issue Dec 10, 2023 · 2 comments · Fixed by #306

Comments

@hyandell
Copy link

Hi Roaster folk,

I noticed that the file https://github.com/forge/roaster/blob/master/tests/src/test/resources/org/jboss/forge/grammar/java/BigInterface.java has an LGPL-2.1 header, which seems out of sorts with the overall Eclipse-2.0 licensing of the rest of the project.

Is this an error, or was it intended?

@gastaldi
Copy link
Member

Hi,

I believe this was copied from some other class that was LGPL, so we just kept the headers. Since it's used in tests only, there isn't a strong reason to change it, but feel free to submit a PR if you think it shouldn't and I'll gladly review it

@hyandell
Copy link
Author

Thanks for the review & reply George.

If Red Hat/JBoss have the permissions to relicense, I'd encourage it to keep things simple. Your point that it's only on the tests is a good one though, so I think it's more about stopping others spending time investigating the licensing oddity.

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 a pull request may close this issue.

2 participants