Openapi-generator: Remove/Update threetenbp dependency

Created on 1 Jul 2019  路  7Comments  路  Source: OpenAPITools/openapi-generator

Description

Please refer to https://github.com/OpenAPITools/openapi-generator/issues/3251#issuecomment-507103814 for more information.

openapi-generator version

Latest master

Suggest a fix/enhancement

Remove the dependency from pom.xml

Compatibility help wanted

All 7 comments

More globally, that would mean support only JDK8+ generation.
Personally I wouldn't mind, we could do a big cleanup by not supporting older JDKs. The only reserve I have is regarding Android support.

Someone familiar with Android development could evaluate using the Java 8 features available in Android Studio. https://developer.android.com/studio/write/java8-support

The probleme is that the java.time package is only available from API 26 and that represents less than 40% of the Android device distribution

What could be removed at least is the supportJava6 option which only represents 3.8% of the distribution of android. And of course, any Android dev should consider moving to Kotlin but I don't know what share of the apps is now written in Kotlin.

Then maybe upgrading the dependency and documenting this as a known issue would be sufficient?

The timezone database embedded in our dependency is from 2015. The artifact's age is probably what triggered the message. But, if people are relying on our code in any way for accurate jsr310 time representations, there are going to be a lot of edge cases that aren't covered due to its age.

It's weird: from what I see most of our builds use threetenbp v1.3.5 which depends on TZDB 2017b

We could upgrade threeten to the latest version btw

Was this page helpful?
0 / 5 - 0 ratings