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

HHH-19188 Remove LGPL-only contributions - Batch 10 #9799

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

marko-bekhta
Copy link
Member

@marko-bekhta marko-bekhta commented Feb 24, 2025

https://hibernate.atlassian.net/browse/HHH-19188


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-19183

@marko-bekhta marko-bekhta force-pushed the fix/HHH-19188-Remove-LGPL-only-contributions---Batch-10 branch 2 times, most recently from 63582b7 to a324556 Compare February 26, 2025 16:32
@gavinking
Copy link
Member

@marko-bekhta We do not need to remove e45e1cf because the author emailed us his consent.

@marko-bekhta
Copy link
Member Author

oook 😃,
do we want to keep the a324556 ?

@gavinking
Copy link
Member

do we want to keep the a324556 ?

No idea, you're the author of that commit.

@marko-bekhta
Copy link
Member Author

No idea, you're the author of that commit.

fair point 😄, I did that to somewhat replace the one you mentioned that we can keep and as a bonus to keep it more consistent between the two impls. In other words, once I revert e45e1cf do you think my change is worth keeping, or the original impl is more easier to read/maintain and I should drop the change?

@gavinking
Copy link
Member

do you think my change is worth keeping, or the original impl is more easier to read/maintain and I should drop the change?

I mean, hard to tell from the diff. Does it reduce the amount of code?

@marko-bekhta marko-bekhta force-pushed the fix/HHH-19188-Remove-LGPL-only-contributions---Batch-10 branch from a324556 to 2fab075 Compare February 27, 2025 09:54
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.

2 participants