[geode-native] branch develop updated: GEODE-5067: Changing visibility preset to 'default' on Clang (#272)

[jb...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jbarrett pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git


The following commit(s) were added to refs/heads/develop by this push:
     new 3694801  GEODE-5067: Changing visibility preset to 'default' on Clang (#272)
3694801 is described below

commit 3694801db067ebcf05a23c5bbe47d8dbdfac512c
Author: Ryan McMahon <rm...@pivotal.io>
AuthorDate: Fri Apr 13 11:24:11 2018 -0700

    GEODE-5067: Changing visibility preset to 'default' on Clang (#272)
    
    - Changing visibility preset to 'default' on Clang to fix RTTI/dynamic_casting issues
    
    Signed-off-by: Ryan McMahon <mc...@gmail.com>
---
 cppcache/shared/CMakeLists.txt | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/cppcache/shared/CMakeLists.txt b/cppcache/shared/CMakeLists.txt
index 6454006..9c29d98 100644
--- a/cppcache/shared/CMakeLists.txt
+++ b/cppcache/shared/CMakeLists.txt
@@ -47,6 +47,11 @@ set_target_properties(apache-geode PROPERTIES
   CXX_VISIBILITY_PRESET hidden
   VISIBILITY_INLINES_HIDDEN ON)
 
+if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
+  set_target_properties(apache-geode PROPERTIES
+    CXX_VISIBILITY_PRESET default)
+endif()
+
 add_dependencies(client-libraries apache-geode)
 
 install(TARGETS apache-geode

-- 
To stop receiving notification emails like this one, please contact
jbarrett@apache.org.