lapps / org.lappsgrid.discriminator

LAPPS discriminators.
Apache License 2.0
0 stars 0 forks source link

Remove unnecessary Groovy dependencies #16

Open reckart opened 5 years ago

reckart commented 5 years ago

Currently, the LAPPS discriminator depends on groovy-all. It surely doesn't need the entire Groovy stack. The dependency footprint and potential for dependency conflicts could be greatly reduced by depending only on those Groovy modules which are in fact used.

ksuderman commented 5 years ago

What version are you using? The groovy-all dependency was removed in 2.3.3. and should have been scoped as a test dependency, that is, <scope>test</scope>, in previous versions.

ksuderman commented 5 years ago

Ok, it is the groovy-parent-pom that includes the dependency on groovy-all, which means all modules will have the same problem. This can likely be reduced to a dependency on just groovy-core.

ksuderman commented 5 years ago

2.4.0-SNAPSHOT has been pushed to the Sonatype snapshot repository. The dependency on groovy-all has been replaced with dependencies on groovy (core) and groovy-xml.