Remove com.sun Dependency

[atkawa7]

No description provided.

[catchex]

Hi, in Java 1.8, tools of package 'com.sun' is required, the ObjectiveSQL minimum supported version is 1.8

[atkawa7]

You can use the alternative javax.lang.model* and javax.tools.* classes instead of calling com.sun packages

[catchex]

What you said should be right, I should given bytecode version while project deploy. But Java 8 users may have problems

[catchex]

I should verify it carefully and release a new version, thanks for you reply.

[atkawa7]

On a side note I was looking at another library that does similar things it seems they do not depend on tools.

https://github.com/google/error-prone/blob/master/check_api/pom.xml
https://github.com/google/error-prone/blob/master/check_api/src/main/java/com/google/errorprone/util/ASTHelpers.java

-        <dependency>
-            <groupId>com.sun</groupId>
-            <artifactId>tools</artifactId>
-            <version>1.0</version>
-            <scope>system</scope>
-            <systemPath>${java.home}/../lib/tools.jar</systemPath>
-            <optional>true</optional>
-        </dependency>

[atkawa7]

Is it possible to see the code that is generated when you inject it via annotation processor?

[catchex]

That's it! because JDK 1.8 is the base version in my project, I can understand your suggestion, this should be a mistake of mine

[atkawa7]

@catchex Did you win?

[catchex]

I had get your suggestion, sorry, codes have not been submitted

[atkawa7]

@catchex ping