From da307f5be2ece998ab30b5ffb66b7dc1ba730c08 Mon Sep 17 00:00:00 2001 From: Theodore Tsirpanis Date: Tue, 22 Oct 2024 10:31:42 +0300 Subject: [PATCH] Fix double installation of `tiledb_serialization.h`. (#5354) This PR fixes `tiledb_serialization.h` being specified for installation twice. We already include the header in `TILEDB_C_API_FILENAME_HEADERS` [here](https://github.com/TileDB-Inc/TileDB/blob/2ff07a56c4a81eec1a986989336c941a9c35690b/CMakeLists.txt#L287-L291), which gets included just above the deleted lines. --- TYPE: BUILD DESC: Fixed double installation of a header when serialization is enabled. --- tiledb/CMakeLists.txt | 6 ------ 1 file changed, 6 deletions(-) diff --git a/tiledb/CMakeLists.txt b/tiledb/CMakeLists.txt index d018eccfdfc..4ecaf58feae 100644 --- a/tiledb/CMakeLists.txt +++ b/tiledb/CMakeLists.txt @@ -69,12 +69,6 @@ set(TILEDB_CORE_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/..") # List of API headers (to be installed) set(TILEDB_PUBLIC_HEADERS ${TILEDB_C_API_FILENAME_HEADERS}) -if (TILEDB_SERIALIZATION) - list(APPEND TILEDB_PUBLIC_HEADERS - ${TILEDB_CORE_INCLUDE_DIR}/tiledb/sm/c_api/tiledb_serialization.h - ) -endif() - if (TILEDB_CPP_API) list(APPEND TILEDB_PUBLIC_HEADERS ${TILEDB_CORE_INCLUDE_DIR}/tiledb/sm/cpp_api/tiledb