ical4j / ical4j-connector

Connectivity with persistent store implementations
http://www.ical4j.org/connector/
Other
9 stars 18 forks source link

Compile-Time Dependency on 'org.codehaus.groovy:groovy-all:2.3.6' #31

Closed stellingsimon closed 6 years ago

stellingsimon commented 6 years ago

We're experiencing problems in our build because ical4j-connector pulls in a compile-time dependency on groovy-all-2.3.6, which is fairly old and incompatible with groovy-all:2.4:

Caused by: groovy.lang.GroovyRuntimeException: Conflicting module versions. Module [groovy-all is loaded in version 2.4.6 and you are trying to load version 2.3.6

A quick search in the ical4j-connector repo didn't reveal an obvious reason why this dependency is needed as a compile-time dependency, but I guess I'm missing something. Or could it be dropped or made a test-only/compile-only dependency instead?

benfortuna commented 6 years ago

Hi Simon, yes i can definitely make it optional dependency at least, will confirm shortly

benfortuna commented 6 years ago

So I removed the groovy dependency completely, I don't think it was required anymore. I've just released 1.0.0 which includes this change, let me know of any issues.