You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by galpal <gi...@git.apache.org> on 2016/06/30 10:03:41 UTC

[GitHub] incubator-geode pull request #179: Feature/geode 88 - adding Makefiles and c...

GitHub user galpal opened a pull request:

    https://github.com/apache/incubator-geode/pull/179

    Feature/geode 88 - adding Makefiles and code fixes for c++ client

    1) adding 3 makefiles to allow compilation of c++ client
    2) linking to ACE dynamically instead of statically
    3) removing "const" and "volatile" to avoid compilation warnings
    4) fixing inconsistencies in memory allocation
    5) adding "vesion.h" and "config.h" files to allow compilation
    6) adding missing include statements to allow compilation
    
    compilation instructions (on linux only):
    In order to build the code on linux using the makefiles that Iam
    submitting,
    You will need to define the following 5 environment variables:
    LIBXML_INCLUDE_DIR
    LIBXML_BIN_DIR
    ACE_THIRD_PARTY_HOME
    INTEL_HOME
    OPTIMIZE_LEVEL
    
    Then run the following command in the following 3 folders:
    Under geode-client-native/src/cppcache:
        > make -f Makefile
    Under geode-client-native/src/sqliteimpl:
        > make -f Makefile
    Under geode-client-native/src/cryptoimpl :
        > make -f Makefile
    
    this will create 3 libraries in the following folders:
        geode-client-native/src/cppcache/build/libgfcppcache.so
        geode-client-native/src/cryptoimpl/build/libcryptoImpl.so
        geode-client-native/src/sqliteimpl/build/libSqLiteImpl.so
    
    you will also see all the object files under the build folder in each module.
    
    To delete the build folders, use:
        > make -f Makefile clean

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/galpal/incubator-geode feature/GEODE-88

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-geode/pull/179.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #179
    
----
commit ac9670005e7a8d3463f9a21247823e32a2501724
Author: Anthony Baker <ab...@apache.org>
Date:   2016-05-13T21:08:53Z

    Add source for geode c++ and .net clients

commit 0029e074dd984d9dea480dc6dbc1080c29cc4a29
Author: galpal <ga...@amdocs.com>
Date:   2016-06-30T09:32:54Z

    GEODE-88: adding Makefiles and code fixes for c++ client
    
    1) adding 3 makefiles to allow compilation of
    2) linking to ACE dynamically instead of statically
    3) removing "const" and "volatile" to avoid compilation warnings
    4) fixing inconsistencies in memory allocation
    5) adding "vesion.h" and "config.h" files to allow compilation
    6) adding missing include statements to allow compilation
    
    compilation instructions (on linux only):
    In order to build the code on linux using the makefiles that Iam submitting,
    You will need to define the following 5 environment variables:
    LIBXML_INCLUDE_DIR
    LIBXML_BIN_DIR
    ACE_THIRD_PARTY_HOME
    INTEL_HOME
    OPTIMIZE_LEVEL
    
    Then run the following command in the following 3 folders:
    Under geode-client-native/src/cppcache:
    	> make -f Makefile
    Under geode-client-native/src/sqliteimpl:
    	> make -f Makefile
    Under geode-client-native/src/cryptoimpl :
    	> make -f Makefile
    
    this will create 3 libraries in the following folders:
    	geode-client-native/src/cppcache/build/libgfcppcache.so
    	geode-client-native/src/cryptoimpl/build/libcryptoImpl.so
    	geode-client-native/src/sqliteimpl/build/libSqLiteImpl.so
    
    you will also see all the object files under the build folder in each module.
    
    To delete the build folders, use:
    	> make -f Makefile clean

commit 08c52fb31723e744b33b8f1baaf5f1c14bf896c1
Author: galpal <ga...@amdocs.com>
Date:   2016-06-30T09:52:47Z

    GEODE-88: adding Makefiles and code fixes for c++ client - new files
    
    1) adding 3 makefiles to allow compilation of
    2) linking to ACE dynamically instead of statically
    3) removing "const" and "volatile" to avoid compilation warnings
    4) fixing inconsistencies in memory allocation
    5) adding "vesion.h" and "config.h" files to allow compilation
    6) adding missing include statements to allow compilation
    
    compilation instructions (on linux only):
    In order to build the code on linux using the makefiles that Iam
    submitting,
    You will need to define the following 5 environment variables:
    LIBXML_INCLUDE_DIR
    LIBXML_BIN_DIR
    ACE_THIRD_PARTY_HOME
    INTEL_HOME
    OPTIMIZE_LEVEL
    
    Then run the following command in the following 3 folders:
    Under geode-client-native/src/cppcache:
        > make -f Makefile
    Under geode-client-native/src/sqliteimpl:
        > make -f Makefile
    Under geode-client-native/src/cryptoimpl :
        > make -f Makefile
    
    this will create 3 libraries in the following folders:
        geode-client-native/src/cppcache/build/libgfcppcache.so
        geode-client-native/src/cryptoimpl/build/libcryptoImpl.so
        geode-client-native/src/sqliteimpl/build/libSqLiteImpl.so
    
    you will also see all the object files under the build folder in each module.
    
    To delete the build folders, use:
        > make -f Makefile clean

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode issue #179: Feature/geode 88 - adding Makefiles and code fix...

Posted by pivotal-jbarrett <gi...@git.apache.org>.
Github user pivotal-jbarrett commented on the issue:

    https://github.com/apache/incubator-geode/pull/179
  
    See CMakeList.txt for production of Makefiles and other project files. All project build files are produced via CMake.
    
    ACE and all other libraries that have native client specific compilation flags and requirements will be statically linked to avoid issues with dynamic linker loading dependencies from system LD_LIBRARY_PATH that do not satisfy specific requirements. 
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request #179: Feature/geode 88 - adding Makefiles and c...

Posted by galpal <gi...@git.apache.org>.
Github user galpal closed the pull request at:

    https://github.com/apache/incubator-geode/pull/179


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode issue #179: Feature/geode 88 - adding Makefiles and code fix...

Posted by galpal <gi...@git.apache.org>.
Github user galpal commented on the issue:

    https://github.com/apache/incubator-geode/pull/179
  
    submitted this pull request by mistake.. I need to add some more changes and then I'll open a new one


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---