Well, that just requires that you appropriately declare dependencies with the "optionality" and "scope" (e.g. "compile", "test", "provided", etc).
Additionally, Geode modules could selectively pull in the required deps as needed. For example, `geode-lucene` would only pull in the Apache Lucene dependencies if the `geode-lucene` module is used. No brainer. On Thu, Feb 28, 2019 at 9:47 AM Charlie Black <cbl...@pivotal.io> wrote: > Hopefully, we are thinking about classpath of the server and lazily adding > these jars only when a feature is turned on. > > On Thu, Feb 28, 2019 at 9:45 AM Dan Smith <dsm...@pivotal.io> wrote: > > > I see that geo, grumpy-core, and commons math came from adding geospatial > > support to redis - > > > > > https://github.com/apache/geode/commit/7bf02251fd047cb1cf575c01b80a9807108618da > > > > -Dan > > > > On Thu, Feb 28, 2019 at 9:41 AM Anthony Baker <aba...@pivotal.io> wrote: > > > > > Looks a number of the new dependencies came in transitively with the > > guava > > > version bump. > > > > > > > On Feb 27, 2019, at 5:32 PM, Anthony Baker <aba...@pivotal.io> > wrote: > > > > > > > > I was reviewing the release branch and noticed a number of new > > > dependencies have been added since the last release. When you add a > new > > > dependency, please review and follow the project license guide [1]. In > > > particular, update the LICENSE file in geode-assembly/src/main/dist > > > depending on the license type. > > > > > > > > Currently we need to update the LICENSE file with the additional > > > MIT/BSD/CDDL dependencies. We may also need to update NOTICE files. > > > There’s also a version conflict with multiple versions of Jackson in > use > > > (2.9.6 / 2.9.8). > > > > > > > > @Sai - these need to be fixed on release/1.9.0 > > > > > > > > Here’s the list of additions: > > > > > > > > animal-sniffer-annotations-1.17.jar > > > > checker-qual-2.5.2.jar > > > > commons-math3-3.2.jar > > > > error_prone_annotations-2.2.0.jar > > > > failureaccess-1.0.jar > > > > geo-0.7.1.jar > > > > grumpy-core-0.2.2.jar > > > > istack-commons-runtime-2.2.jar > > > > j2objc-annotations-1.1.jar > > > > javax.activation-1.2.0.jar > > > > javax.activation-api-1.2.0.jar > > > > jsr305-3.0.2.jar > > > > listenablefuture-9999.0-empty-to-avoid-conflict-with-guava.jar > > > > > > > > Removed: > > > > > > > > activation-1.1.1 > > > > jaxb-core-2.2.11.jar > > > > > > > > Anthony > > > > > > > > [1] > > > > > > https://cwiki.apache.org/confluence/display/GEODE/License+Guide+for+Contributors > > > < > > > > > > https://cwiki.apache.org/confluence/display/GEODE/License+Guide+for+Contributors > > > > > > > > > > > > > > > > > > > -- > Charlie Black | cbl...@pivotal.io > -- -John john.blum10101 (skype)
