Skip to content

Commit ea86bf8

Browse files
committed
handle Nabla & DXC package keys with INTERFACE to expose in defines header
1 parent 1ed9234 commit ea86bf8

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

cmake/common.cmake

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,8 +157,6 @@ macro(nbl_create_executable_project _EXTRA_SOURCES _EXTRA_OPTIONS _EXTRA_INCLUDE
157157
target_compile_definitions(${EXECUTABLE_NAME}
158158
PRIVATE "-DNBL_CPACK_PACKAGE_NABLA_DLL_DIR=\"${_NBL_NABLA_PACKAGE_RUNTIME_DLL_DIR_PATH_REL_TO_TARGET_}\""
159159
PRIVATE "-DNBL_CPACK_PACKAGE_DXC_DLL_DIR=\"${_NBL_DXC_PACKAGE_RUNTIME_DLL_DIR_PATH_REL_TO_TARGET_}\""
160-
PRIVATE "-DNBL_CPACK_PACKAGE_NABLA_DLL_DIR_ABS_KEY=\"${_NBL_NABLA_PACKAGE_RUNTIME_DLL_DIR_PATH_}\""
161-
PRIVATE "-DNBL_CPACK_PACKAGE_DXC_DLL_DIR_ABS_KEY=\"${_NBL_DXC_PACKAGE_RUNTIME_DLL_DIR_PATH_}\""
162160
)
163161
endif()
164162

src/nbl/CMakeLists.txt

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -364,8 +364,8 @@ if(NBL_CPACK_NO_BUILD_DIRECTORY_MODULES)
364364
endif()
365365

366366
target_compile_definitions(Nabla
367-
PUBLIC _DXC_DLL_="${DXC_DLL}"
368-
PUBLIC _DXC_DLL_NAME_="$<PATH:REMOVE_EXTENSION,${DXC_DLL_NAME}>"
367+
INTERFACE _DXC_DLL_="${DXC_DLL}"
368+
INTERFACE _DXC_DLL_NAME_="$<PATH:REMOVE_EXTENSION,${DXC_DLL_NAME}>"
369369
PRIVATE __NBL_BUILDING_NABLA__
370370
)
371371

@@ -393,8 +393,9 @@ endif()
393393
set(NBL_ASSEMBLY_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}/$<CONFIG>/devshgraphicsprogramming.nabla")
394394
if(NOT NBL_STATIC_BUILD)
395395
set_target_properties(Nabla PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${NBL_ASSEMBLY_DIRECTORY})
396-
target_compile_definitions(Nabla PUBLIC
397-
_NABLA_DLL_NAME_="$<PATH:REMOVE_EXTENSION,$<TARGET_FILE_NAME:Nabla>>";_NABLA_OUTPUT_DIR_="${NBL_ASSEMBLY_DIRECTORY}"
396+
target_compile_definitions(Nabla
397+
INTERFACE _NABLA_DLL_NAME_="$<PATH:REMOVE_EXTENSION,$<TARGET_FILE_NAME:Nabla>>"
398+
INTERFACE _NABLA_OUTPUT_DIR_="${NBL_ASSEMBLY_DIRECTORY}"
398399
)
399400
endif()
400401

@@ -679,7 +680,7 @@ if(NBL_EMBED_BUILTIN_RESOURCES)
679680
endif()
680681

681682
target_compile_definitions(Nabla
682-
PUBLIC _NABLA_INSTALL_DIR_="${CMAKE_INSTALL_PREFIX}"
683+
INTERFACE _NABLA_INSTALL_DIR_="${CMAKE_INSTALL_PREFIX}"
683684
)
684685

685686
set(DEFINE_H "${CMAKE_CURRENT_BINARY_DIR}/include/$<CONFIG>/define.h")
@@ -789,4 +790,11 @@ endif()
789790
nbl_install_program_spec("${DXC_DLL}" "nbl/3rdparty/dxc")
790791
nbl_install_dir_spec(../../include/nbl/application_templates nbl)
791792

793+
# note: keep after install rules due to NBL_3RDPARTY_DXC_NS_PACKAGE_RUNTIME_DLL_DIR_PATH property
794+
get_property(_NBL_DXC_PACKAGE_RUNTIME_DLL_DIR_PATH_ GLOBAL PROPERTY NBL_3RDPARTY_DXC_NS_PACKAGE_RUNTIME_DLL_DIR_PATH)
795+
target_compile_definitions(Nabla
796+
INTERFACE NBL_CPACK_PACKAGE_NABLA_DLL_DIR_ABS_KEY="$<GENEX_EVAL:$<TARGET_PROPERTY:Nabla,NBL_PACKAGE_RUNTIME_DLL_DIR_PATH>>"
797+
INTERFACE NBL_CPACK_PACKAGE_DXC_DLL_DIR_ABS_KEY="${_NBL_DXC_PACKAGE_RUNTIME_DLL_DIR_PATH_}"
798+
)
799+
792800
NBL_ADJUST_FOLDERS(src)

0 commit comments

Comments
 (0)