[thrift] branch master updated: THRIFT-5086 CMake target thrift::thrift has no INTERFACE_INCLUDE_DIRECTORIES property Client: cpp Patch: SmartNet Club

[jensg]

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

jensg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new baddaa8  THRIFT-5086 CMake target thrift::thrift has no 
INTERFACE_INCLUDE_DIRECTORIES property Client: cpp Patch: SmartNet Club
baddaa8 is described below

commit baddaa8612fe70070208d30b91d820e99fb2a716
Author: Jens Geyer <je...@apache.org>
AuthorDate: Wed Feb 5 22:30:01 2020 +0100

    THRIFT-5086 CMake target thrift::thrift has no 
INTERFACE_INCLUDE_DIRECTORIES property
    Client: cpp
    Patch: SmartNet Club
    
    This closes #2003
---
 build/cmake/ThriftMacros.cmake | 1 +
 1 file changed, 1 insertion(+)

diff --git a/build/cmake/ThriftMacros.cmake b/build/cmake/ThriftMacros.cmake
index d068b2a..038651e 100644
--- a/build/cmake/ThriftMacros.cmake
+++ b/build/cmake/ThriftMacros.cmake
@@ -25,6 +25,7 @@ endmacro(ADD_PKGCONFIG_THRIFT)
 
 macro(ADD_LIBRARY_THRIFT name)
     add_library(${name} ${ARGN})
+    target_include_directories(${name} INTERFACE $<INSTALL_INTERFACE:include>)
     set_target_properties(${name} PROPERTIES
         OUTPUT_NAME ${name}${THRIFT_RUNTIME_POSTFIX}   # windows link variants 
(/MT, /MD, /MTd, /MDd) get different names
         VERSION ${thrift_VERSION} )