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 )

*OPTION 2*
Add Geode Jars as external dependency  and update CMakeFile accordingly to
download as done for other dependencies


Thanks
Avinash

[1]
http://www-eu.apache.org/dist/geode/1.0.0-incubating/apache-geode-1.0.0-incubating.tar.gz


On Sat, Jan 14, 2017 at 4:30 AM, Anthony Baker <aba...@apache.org> wrote:

> I am pleased to announce the donation of improved GemFire client
> drivers to the Geode community. This source code donation includes a
> C++ client and a .NET client.  This new grant attempts to greatly
> improve the mergability of the code base making it friendlier to the
> community by providing an improved build system, better source code
> organization, and many additional fixes. We hope that it will be much
> more accessible for the project community and will lead to a much
> quicker integration into an overall codebase (unlike its predecessor
> [1]).”
>
> The C++ client allows an application to connect to a cluster using
> intelligent network routing for low latency reads and writes.  The C++
> client can additionally cache data locally and receive update
> notifications or register continuous queries.  The .NET client wraps
> the C++ libraries and enables .NET applications to be written for
> Geode.  Both of these clients are functionally equivalatent to the
> already donated Java client for Geode.
>
> The Software Grant Agreement for this code has been accepted by the
> ASF secretary.
>
> The donated code currently sits in a separate branch in the Geode
> repository named next-gen-native-client-software-grant [2] and is
> awaiting community review.  I encourage everyone in the Geode
> community to review this donation and provide feedback.  In particular
> your input on build improvements would be really helpful.  Once the
> community has reached a consensus we can determine next steps and how
> this code might get merged into the develop branch [3] so that all
> users can access these features.  Your suggestions are most welcome!
>
>
> Thanks,
> Anthony
>
> [1] http://mail-archives.apache.org/mod_mbox/geode-dev/201605.
> mbox/%3cCAEwge-ENhF4s_k5Y=h9-mNFANb777j8bNDe4U9jahPYim61pfg
> @mail.gmail.com%3e
> [2] https://git-wip-us.apache.org/repos/asf?p=geode.git;a=
> shortlog;h=refs/heads/next-gen-native-client-software-grant
> [3] https://issues.apache.org/jira/browse/GEODE-1416
>

Reply via email to