geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kirk Lund <kl...@apache.org>
Subject Re: build is broken
Date Tue, 23 Jan 2018 19:15:15 GMT
I have it working now. My IntelliJ project was using JDK1.8.0_66 for Gradle
which prevented updating of dependencies which then caused compilation in
IntelliJ to fail on that new dependency.

This doesn't resolve my main issue here which is this: *all new
dependencies should be discussed on the dev list before introducing them in
a commit*. Can you please point me at an email that proposes adding this
dependency?

On Tue, Jan 23, 2018 at 10:10 AM, Anthony Baker <abaker@pivotal.io> wrote:

> I just ran `gradle clean build` from HEAD and did not get any errors.
>
> > On Jan 23, 2018, at 10:08 AM, Jens Deppe <jdeppe@pivotal.io> wrote:
> >
> > The dependency is the rmi-io library. It provides functionality for
> > streaming data over RMI.
> >
> > It was added a few weeks ago, so I'm not sure why your build is breaking
> > now.
> >
> > --Jens
> >
> > On Tue, Jan 23, 2018 at 9:41 AM, Kirk Lund <klund@apache.org> wrote:
> >
> >> Looks like this dependency is Jackcess (used for reading and writing MS
> >> Access databases). I cannot find any discussion about adding this
> >> dependency on the dev-list. Why does Geode need to talk to MS Access
> >> databases for Cluster Configuration?
> >>
> >> It does have an Apache 2.0 license, but I think there are two problems
> with
> >> adding Jackcess: 1) all new dependencies should be discussed on dev-list
> >> BEFORE adding them to Geode, 2) we shouldn't need to talk to MS Access
> for
> >> Cluster Configuration.
> >>
> >> On Tue, Jan 23, 2018 at 9:30 AM, Kirk Lund <klund@pivotal.io> wrote:
> >>
> >>> I just tried to compile Geode develop. ClusterConfigurationService has
> >> two
> >>> broken imports that don't exist:
> >>>
> >>> import com.healthmarketscience.rmiio.RemoteInputStream;
> >>> import com.healthmarketscience.rmiio.RemoteInputStreamClient;
> >>>
> >>> Was there a new dependency added to Geode?
> >>>
> >>
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message