Support Java 8 time about cron-utils HOT 12 CLOSED

@JensWalter thanks for forking cron-utils! Would you mind merging forked changes into a new branch?

from cron-utils.

Sure, but so far I didn't came to the point where I could actually test my implementation. So far I only got to the point where it compiles.
So I could merge it back, but the current status is rather experimental. So should I merge back or first try to test a little more?

from cron-utils.

If current status is experimental, I would prefer if you can test it more. Quality matters!
Can I assign the issue to you? Thanks!

from cron-utils.

Once we have this branch, it would be great to setup different classifiers, to produce compatible versions for different JDKs. A resource on this here

from cron-utils.

@JensWalter Hi! are you still willing to migrate this project to JDK8?
If doing so, it would be an opportunity to get rid of guava, since there were some complaints on how heavy that dependency is, and in JDK8 we have full support for Functions.

from cron-utils.

We are migrating cron-utils to JDK8. This task will be developed on branch jdk8
With this migration effort we are also removing guava dependency, in order to avoid unnecessary dependencies on libraries.

from cron-utils.

Migration to JDK8 was done. Replacing JodaTime for Java Time remains.

from cron-utils.

@jmrozanec
For replacing JodaTime with Java 8's API do we want to migrate all methods that return a joda time "DateTime" object with Java 8's "LocalDateTime" or "ZonedDateTime" ?

I'm leaning towards "LocalDateTime". Just wondering what are your thoughts on this matter.

from cron-utils.

@espiegel we are willing to replace all methods regarding Jodatime "DateTime". This would be a major change. I would lean towards ZonedDateTime, since better reflects behavior "DateTime" provides us today. Note that there are several cases, where DateTime gives us a notion of timezone where the time was being used.

from cron-utils.

@jmrozanec Sounds good, I'll get started on that later this week.

from cron-utils.

@espiegel Great! Looking forward to merge the changes!

from cron-utils.

@JensWalter The feature was contributed by @espiegel at PR#107 Thank you @espiegel for tackling this!

from cron-utils.