Merge pull request #15 from LLNL/task/2016_07_install_builtin_tpls

bugfix: reenable install of libcivet and libbb64
LLNL · Jul 30, 2016 · cc29d5f · cc29d5f
2 parents 85e1e05 + 3b5808f
commit cc29d5f
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/src/CMake/CMakeBasics.cmake b/src/CMake/CMakeBasics.cmake
@@ -168,7 +168,16 @@ macro(add_compiled_library)
     # install and export setup
     #############################
     # if export is set, install our lib
-    if(NOT ${args_INTERNAL} OR WIN32)
+
+    #
+    # See https://github.com/LLNL/conduit/issues/14
+    # we are installing these (for now) due to an issue other cmake
+    # build systems importing conduit targets and still needing
+    # to link these type of libs, even though we are using them
+    # with PRIVATE cmake link properties.
+    #
+    #if(NOT ${args_INTERNAL} OR WIN32)
+    #
 
         # if we have headers, install them
         if(NOT "${args_HEADERS}" STREQUAL "")
@@ -192,7 +201,7 @@ macro(add_compiled_library)
                     ARCHIVE DESTINATION lib
                     RUNTIME DESTINATION lib)
         endif()
-    endif()
+    #endif()
 
 endmacro()