Skip to content

Commit da307f5

Browse files
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.
1 parent 26610b0 commit da307f5

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

tiledb/CMakeLists.txt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -69,12 +69,6 @@ set(TILEDB_CORE_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/..")
6969
# List of API headers (to be installed)
7070
set(TILEDB_PUBLIC_HEADERS ${TILEDB_C_API_FILENAME_HEADERS})
7171

72-
if (TILEDB_SERIALIZATION)
73-
list(APPEND TILEDB_PUBLIC_HEADERS
74-
${TILEDB_CORE_INCLUDE_DIR}/tiledb/sm/c_api/tiledb_serialization.h
75-
)
76-
endif()
77-
7872
if (TILEDB_CPP_API)
7973
list(APPEND TILEDB_PUBLIC_HEADERS
8074
${TILEDB_CORE_INCLUDE_DIR}/tiledb/sm/cpp_api/tiledb

0 commit comments

Comments
 (0)