Skip to content

Conversation

@staticlibs
Copy link
Collaborator

This PR adds the name duckdb.jdbc as an automatic JPMS module name to the driver JAR. This makes this name, that was previously derived from the JAR file name, to be stable when the driver is used from modular applications.

Testing: CI test added that checks the module name.

Fixes: #431

This PR adds the name `duckdb.jdbc` as an [automatic JPMS module](https://openjdk.org/projects/jigsaw/spec/sotms/#automatic-modules)
name to the driver JAR. This makes this name, that was previously
derived from the JAR file name, to be stable when the driver is used
from modular applications.

Testing: CI test added that checks the module name.

Fixes: duckdb#431
@staticlibs staticlibs merged commit 694d3a3 into duckdb:main Nov 10, 2025
10 checks passed
@staticlibs staticlibs deleted the jpms_name branch November 10, 2025 14:41
staticlibs added a commit to staticlibs/duckdb-java that referenced this pull request Nov 10, 2025
This is a backport of the PR duckdb#454 to `v1.4-andium` stable branch.

This PR adds the name `duckdb.jdbc` as an [automatic JPMS module](https://openjdk.org/projects/jigsaw/spec/sotms/#automatic-modules)
name to the driver JAR. This makes this name, that was previously
derived from the JAR file name, to be stable when the driver is used
from modular applications.

Testing: CI test added that checks the module name.

Fixes: duckdb#431
staticlibs added a commit that referenced this pull request Nov 10, 2025
This is a backport of the PR #454 to `v1.4-andium` stable branch.

This PR adds the name `duckdb.jdbc` as an [automatic JPMS module](https://openjdk.org/projects/jigsaw/spec/sotms/#automatic-modules)
name to the driver JAR. This makes this name, that was previously
derived from the JAR file name, to be stable when the driver is used
from modular applications.

Testing: CI test added that checks the module name.

Fixes: #431
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.

Add module info to duckdb_jdbc

1 participant