bjornharrtell / jts2geojson

JTS from/to GeoJSON converter for Java
MIT License
141 stars 57 forks source link

Downgrade jdk version to 1.7 #30

Closed gorshkov-leonid closed 5 years ago

gorshkov-leonid commented 5 years ago

Hello, a lot of customers use jdk 1.7. So we also have to choose libraries that compiled for jdk 7 or publish downgraded fork. I think there isn't reason to use jdk 1.7 in this project, because it was built successfully with options 1.7 in maven-compiler-plugin. Could you downgrade version of java source/target in maven compiler plugin or release a special version for us? It will be better rather then using with changed coordinates.

bjornharrtell commented 5 years ago

Not interested in supporting JDK 1.7.

gorshkov-leonid commented 5 years ago

Ок

gorshkov-leonid commented 5 years ago

How many problems did you encounter due to the class version change?

bjornharrtell commented 5 years ago

I don't have any problems. :)

gorshkov-leonid commented 5 years ago

I predicted that answer) sorry for bother :) But I thought if a library uses required dependencies instead of latest ones then it can be used by more people. So, jts2geojson can use java 1.7 and jackson 2.7.x. If you don't want ... Well, there is no way. Thank you for answer

bjornharrtell commented 5 years ago

I think people should use at least the lowest supported JDK, or use older releases of deps if available. This project is on low maintenance for me so I really don't want to bother with unsupported platforms.

gorshkov-leonid commented 5 years ago

Got it, thank you.