sendgrid-java icon indicating copy to clipboard operation
sendgrid-java copied to clipboard

Client incompatible with JPMS modules

Open chrylis opened this issue 4 years ago • 5 comments

This is a reopen request for #440 and #566, which have not been fixed. Specifically, since you provide classes in the com.sendgrid namespace from multiple jars, it is not possible to use your client from the modulepath.

JPMS went GA four years ago and was available in preview for some time before that, and Java 11 is about to be replaced as the LTS baseline by Java 17. Getting the SendGrid client working with JPMS will not be overly difficult, and it's a blocker for some of your users.

chrylis avatar Aug 19 '21 01:08 chrylis

Hi @chrylis I've reopened issue #440. Sendgrid-java already supports Java 11. Other than issue #440 and providing classes in the com.sendgrid namespace from multiple jars, are there any other blockers preventing use of JPMS?

JenniferMah avatar Aug 24 '21 22:08 JenniferMah

Error occurred during initialization of boot layer java.lang.module.ResolutionException: Modules java.http.client and sendgrid.java export package com.sendgrid to module org.postgresql.jdbc

chrisjenx avatar Sep 27 '21 02:09 chrisjenx

Create a dist actually compiled against JDK 11 :roll_eyes:

chrisjenx avatar Sep 27 '21 02:09 chrisjenx

Hi @chrylis I've reopened issue #440. Sendgrid-java already supports Java 11. Other than issue #440 and providing classes in the com.sendgrid namespace from multiple jars, are there any other blockers preventing use of JPMS?

Just to clarify, the current jar is incompatible with JDK11+ built applications. Many libs bumped a major version to do this, in your case, 4.x could continue as J8 then a 5.x could target JDK11+.

See https://github.com/brettwooldridge/HikariCP for example

chrisjenx avatar Sep 27 '21 02:09 chrisjenx

Bump

chrisjenx avatar Nov 02 '21 15:11 chrisjenx