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

[CONJ-1053] Make the JPMS requirement on waffle-jna static #185

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

Conversation

A248
Copy link

@A248 A248 commented May 14, 2023

The waffle-jna dependency is needed only when Windows GSSAPI authentication is used. In a previous commit, the static was incorrectly removed, making the dependency a hard requirement when running on JPMS.

5ac67b2

However, maven is not the same as JPMS. An optional dependency via Maven is not the same as requires static in JPMS. They serve entirely different purposes. By removing requires static, you are saying that all deployments must have waffle-jna present on the module path.

Re-adding requires static will not affect existing clients using MariaDB Connector-J. If they used waffla-jna before, they will continue to use it after this change.

https://jira.mariadb.org/browse/CONJ-1053

@piewie
Copy link

piewie commented Sep 15, 2023

@A248 any reason why this is not merged yet? We stumble on this issue

@A248
Copy link
Author

A248 commented Sep 15, 2023

@piewie I'm not a maintainer. You could politely ask Diego Dupin (@rusher) or comment on the linked JIRA issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants