search

jglick / sezpoz

SezPoz: lightweight, annotation-based service loader
29 stars 11 forks source link

sezpoz doesn't compile with Java 8 #8

Closed ebourg closed 9 years ago

ebourg commented 10 years ago

Hi,

SezPoz doesn't compile with Java 8 due to the removal of APT. The compiler complains that com.sun.mirror.declaration and com.sun.mirror.apt are missing. The javax.annotation.processing API should be used instead.

jglick commented 9 years ago

Was fixed long ago in ab326d4c243e058f30479f597ab525f32076168b.

ebourg commented 9 years ago

Thank you Jesse. Any plan to release a new version with this modification by chance?

jglick commented 9 years ago

I could, though I am curious why you care—the binaries of the last release should work fine on Java 8, and if you are building from sources probably you are interested in the latest master branch.

ebourg commented 9 years ago

I'm maintaining the sezpoz package in Debian and I have to be able to build from sources with Java 8 (for now the package is built with Java 7, but it will go away in Debian 9 and Java 8 will then be used by default). If a new release is expected in a few months I can wait a bit, otherwise I'll just cherry pick the modification.

jglick commented 9 years ago

Ah, that makes more sense. Reviewing unreleased changes…

jglick commented 9 years ago

OK, 1.10 released, should make it to Central in a few hours.

ebourg commented 9 years ago

Thank you very much, this really helps.