fix cmake

Author: lmangani

chsql/CMakeLists.txt

@@ -1,5 +1,7 @@
 cmake_minimum_required(VERSION 3.12)
 
+set(CMAKE_CXX_STANDARD 17)
+
 # Set extension name here
 set(TARGET_NAME chsql)
 
@@ -28,14 +30,6 @@ include_directories(
     ./src/include
     ./src
     ${CMAKE_CURRENT_SOURCE_DIR}/../duckdb/extension/parquet/include
-    ../duckdb/third_party/lz4
-    ../duckdb/third_party/parquet
-    ../duckdb/third_party/thrift
-    ../duckdb/third_party/snappy
-    ../duckdb/third_party/zstd/include
-    ../duckdb/third_party/mbedtls
-    ../duckdb/third_party/mbedtls/include
-    ../duckdb/third_party/brotli/include
     ${CMAKE_CURRENT_SOURCE_DIR}/../contribs/clickhouse-cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/../contribs/clickhouse-cpp/contrib/absl
 )
@@ -51,13 +45,20 @@ build_static_extension(${TARGET_NAME} ${EXTENSION_SOURCES})
 build_loadable_extension(${TARGET_NAME} " " ${EXTENSION_SOURCES})
 
 # Link libraries using plain signature
-target_link_libraries(${EXTENSION_NAME} OpenSSL::SSL OpenSSL::Crypto clickhouse-cpp-lib absl_int128 cityhash lz4 zstdstatic)
-target_link_libraries(${LOADABLE_EXTENSION_NAME} OpenSSL::SSL OpenSSL::Crypto clickhouse-cpp-lib absl_int128 cityhash lz4 zstdstatic)
+target_link_libraries(${EXTENSION_NAME} OpenSSL::SSL OpenSSL::Crypto clickhouse-cpp-lib)
+target_link_libraries(${LOADABLE_EXTENSION_NAME} OpenSSL::SSL OpenSSL::Crypto clickhouse-cpp-lib)
 
 # Install targets
 install(
-    TARGETS ${EXTENSION_NAME}
+    TARGETS ${EXTENSION_NAME} clickhouse-cpp-lib absl_int128 cityhash lz4 zstdstatic
     EXPORT "${DUCKDB_EXPORT_SET}"
     LIBRARY DESTINATION "${INSTALL_LIB_DIR}"
     ARCHIVE DESTINATION "${INSTALL_LIB_DIR}"
 )
+
+# Make sure the export set includes clickhouse-cpp-lib and its dependencies
+export(
+    TARGETS ${EXTENSION_NAME} clickhouse-cpp-lib absl_int128 cityhash lz4 zstdstatic
+    NAMESPACE duckdb::
+    FILE DuckDBExports.cmake
+)