From 0eb6d18c84a356be94e44c7043d4a9ae02033e4c Mon Sep 17 00:00:00 2001 From: Matthew Fernandez Date: Wed, 6 Apr 2022 18:38:10 -0700 Subject: [PATCH] CMake: fix: use a relative 'DATA_INSTALL_DIR' on non-Linux Some installers like NSIS on Windows do not allow absolute paths in installation destinations. This is already followed in e.g. `BINARY_INSTALL_DIR`, but `DATA_INSTALL_DIR` was using an absolute path. This issue was not detected because this is currently only used in Smyrna which is not enabled on Windows in CI. But an upcoming commit enables general example installation, which exposes this issue. Gitlab: #1836 --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 28e8283a0..671a5d297 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -74,7 +74,7 @@ else() set(MAN_INSTALL_DIR share/man) set(libdir "${CMAKE_INSTALL_PREFIX}/${LIBRARY_INSTALL_DIR}") set(includedir "${CMAKE_INSTALL_PREFIX}/include") - set(DATA_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/share") + set(DATA_INSTALL_DIR share) endif() set(PLUGIN_INSTALL_DIR ${LIBRARY_INSTALL_DIR}/graphviz) -- 2.50.1