Thanks Jacob.
Build goes fine with this change.
Thanks
Avinash
On Tue, Jan 17, 2017 at 5:17 AM, Jacob Barrett <jbarr...@pivotal.io> wrote:
> Avinash,
>
> We just added some changes to the Geode dependency. You should be able to
> set cmake -DGEODE_ROOT=/path/to/apache-geode. Without that flag set it
> will
> look for Geode in:
>
> /geode/bin
> /apache-geode/bin
> /usr/geode/bin
> /usr/apache-geode/bin
> /usr/local/geode/bin
> /usr/local/apache-geode/bin
> /opt/geode/bin
> /opt/apache-geode/bin
> /opt/local/geode/bin
> /opt/local/apache-geode/bin
>
>
> -Jake
>
>
> On Sun, Jan 15, 2017 at 2:24 AM Avinash Dongre <adon...@apache.org> wrote:
>
> > This is great.
> >
> > Need to make following change to PASS the build.
> >
> >
> > diff --git a/src/tests/javaobject/CMakeLists.txt
> > b/src/tests/javaobject/CMakeLists.txt
> > index 4924e5c..7f85878 100644
> > --- a/src/tests/javaobject/CMakeLists.txt
> > +++ b/src/tests/javaobject/CMakeLists.txt
> > @@ -9,8 +9,8 @@ get_filename_component(JAVA_HOME ${JAVA_BIN} DIRECTORY)
> >
> > # Update the class path.
> > set(CMAKE_JAVA_INCLUDE_PATH ${JAVA_HOME}/lib/tools.jar)
> > -if (GEMFIRE_HOME)
> > - LIST(APPEND CMAKE_JAVA_INCLUDE_PATH
> > ${GEMFIRE_HOME}/lib/geode-core-9.0.0.jar)
> > +if (DEFINED ENV{GEMFIRE_HOME})
> > + LIST(APPEND CMAKE_JAVA_INCLUDE_PATH
> > $ENV{GEMFIRE_HOME}/lib/geode-core-1.0.0-incubating.jar)
> > else()
> > LIST(APPEND CMAKE_JAVA_INCLUDE_PATH
> > "/gemfire/lib/geode-core-9.0.0.jar")
> > endif()
> >
> >
> > Also need to update the src/BUILDING.md for instruction to download geode
> > build from [1] and set GEMFIRE_HOME ( which I think should be changed to
> > GEODE_HOME )
> >
> >
>
