Skip to content

Commit ff7ce0e

Browse files
committed
Small corrections, let's prepare it for the case microsoft restrict less probing
1 parent 4c8b666 commit ff7ce0e

File tree

4 files changed

+14
-7
lines changed

4 files changed

+14
-7
lines changed

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ include(${CMAKE_CURRENT_SOURCE_DIR}/cmake/toolchains/android/build.cmake)
1515

1616
project(Nabla LANGUAGES CXX C)
1717

18-
option(NBL_STATIC_BUILD "" OFF) # ON for static builds, OFF for shared
19-
option(NBL_DYNAMIC_MSVC_RUNTIME "" ON)
18+
option(NBL_STATIC_BUILD "" ON) # ON for static builds, OFF for shared
19+
option(NBL_DYNAMIC_MSVC_RUNTIME "" OFF)
2020

2121
if(NBL_STATIC_BUILD)
2222
message(STATUS "Static Nabla build enabled!")

cmake/common.cmake

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ macro(nbl_create_executable_project _EXTRA_SOURCES _EXTRA_OPTIONS _EXTRA_INCLUDE
4949
VERBATIM
5050
)
5151

52-
add_custom_target(${EXECUTABLE_NAME}_config ALL DEPENDS ${NBL_CONFIG_OUTPUT_FILE} ${NBL_ROOT_PATH}/cmake/scripts/nbl/applicationMSVCConfig.cmake)
52+
add_custom_target(${EXECUTABLE_NAME}_with_config ALL DEPENDS ${NBL_CONFIG_OUTPUT_FILE} ${NBL_ROOT_PATH}/cmake/scripts/nbl/applicationMSVCConfig.cmake)
5353
endif()
5454

5555
set(NBL_EXECUTABLE_SOURCES
@@ -58,7 +58,6 @@ macro(nbl_create_executable_project _EXTRA_SOURCES _EXTRA_OPTIONS _EXTRA_INCLUDE
5858
)
5959

6060
add_executable(${EXECUTABLE_NAME} ${NBL_EXECUTABLE_SOURCES})
61-
add_dependencies(${EXECUTABLE_NAME} Nabla_manifest)
6261

6362
if(NBL_DYNAMIC_MSVC_RUNTIME)
6463
set_property(TARGET ${EXECUTABLE_NAME} PROPERTY MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>DLL")
@@ -68,7 +67,13 @@ macro(nbl_create_executable_project _EXTRA_SOURCES _EXTRA_OPTIONS _EXTRA_INCLUDE
6867
endif()
6968

7069
# EXTRA_SOURCES is var containing non-common names of sources (if any such sources, then EXTRA_SOURCES must be set before including this cmake code)
71-
add_dependencies(${EXECUTABLE_NAME} Nabla)
70+
if(NBL_STATIC_BUILD)
71+
add_dependencies(${EXECUTABLE_NAME} Nabla)
72+
else()
73+
add_dependencies(${EXECUTABLE_NAME}_with_config Nabla_with_manifest)
74+
#target_link_options(${EXECUTABLE_NAME} PRIVATE "/manifestdependency:\"type='win32' name='devshgraphicsprogramming.nabla' version='1.2.3.4' processorArchitecture='x86' language='*'\"")
75+
endif()
76+
7277
get_target_property(NBL_EGL_INCLUDE_DIRECORIES egl INCLUDE_DIRECTORIES)
7378

7479
target_include_directories(${EXECUTABLE_NAME}

cmake/scripts/nbl/applicationMSVCConfig.cmake

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ set(NBL_INPUT_MSVC_CONFIG_FILE ${NBL_ROOT_PATH}/cmake/config/msvc/application.ex
4040

4141
file(RELATIVE_PATH NBL_DLL_RELATIVE_PATH "${NBL_GEN_DIRECTORY}" "${NBL_DLL_PATH}")
4242
cmake_path(GET NBL_DLL_RELATIVE_PATH PARENT_PATH NBL_ASSEMLBY_RELATIVE_PATH)
43+
cmake_path(GET NBL_ASSEMLBY_RELATIVE_PATH PARENT_PATH NBL_ASSEMLBY_RELATIVE_PATH)
4344

4445
message(STATUS "Generating ${NBL_GEN_DIRECTORY}/${NBL_TARGET_NAME}.config")
4546
configure_file("${NBL_INPUT_MSVC_CONFIG_FILE}" "${NBL_GEN_DIRECTORY}/${NBL_TARGET_NAME}.config")

src/nbl/CMakeLists.txt

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -457,13 +457,14 @@ endif()
457457

458458
if(NOT NBL_STATIC_BUILD)
459459
set(NBL_ASSEMBLY_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}/$<CONFIG>/devshgraphicsprogramming.nabla" CACHE INTERNAL "" FORCE)
460+
set(NBL_ASSEMBLIES_MANIFEST_FILEPATH "${NBL_ASSEMBLY_DIRECTORY}/devshgraphicsprogramming.nabla.manifest" CACHE INTERNAL "" FORCE)
460461
add_custom_command(OUTPUT "${NBL_ASSEMBLIES_MANIFEST_FILEPATH}"
461462
COMMAND ${CMAKE_COMMAND} -DNBL_ROOT_PATH:PATH=${NBL_ROOT_PATH} -DNBL_GEN_DIRECTORY:PATH=${NBL_ASSEMBLY_DIRECTORY} -DNABLA_DLL_PATH:FILEPATH=$<TARGET_FILE:Nabla> -P ${NBL_ROOT_PATH}/cmake/scripts/nbl/nablaMSVCManifest.cmake
462463
COMMENT "Launching nabla.manifest generation script!"
463464
VERBATIM
464465
)
465-
466-
add_custom_target(Nabla_manifest ALL DEPENDS ${NBL_ASSEMBLIES_MANIFEST_FILEPATH} ${NBL_ROOT_PATH}/cmake/scripts/nbl/nablaMSVCManifest.cmake)
466+
467+
add_custom_target(Nabla_with_manifest ALL DEPENDS ${NBL_ASSEMBLIES_MANIFEST_FILEPATH})
467468
set_target_properties(Nabla PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${NBL_ASSEMBLY_DIRECTORY})
468469
endif()
469470

0 commit comments

Comments
 (0)