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

Reply via email to