Organise output binaries

Improve build and debugger experience in VS.

Author: twostars

CMakeLists.txt

@@ -9,6 +9,9 @@ project(OpenKO
 option(OPENKO_BUILD_SERVERS "Build the server projects (default on)" ON)
 option(OPENKO_COMPILE_WARNINGS_AS_ERROR "Enable treating warnings as errors (default on)" ON)
 
+# Define the bin directory; defaults to repo/bin
+set(OPENKO_BIN_DIR "${CMAKE_BINARY_DIR}/bin" CACHE PATH "Directory for final binaries")
+
 if(WIN32)
   # Set the minimum Windows version
   set(_WIN32_WINNT "0x0A00" CACHE STRING "Minimum Windows version (default 0x0A00 - Windows 10)")
@@ -17,9 +20,22 @@ endif()
 
 set(SPDLOG_ENABLE_PCH ON CACHE BOOL "Enable PCH in spdlog")
 set(NANODBC_FORCE_LIBCXX OFF CACHE BOOL "Force the use of libc++ (default off)")
-set(NANODBC_DISABLE_MSSQL_TVP ON "Disable MSSQL Table-valued parameters in nanodbc")
-set(NANODBC_DISABLE_NULL_ACCESS_ERROR ON "Disable null_access_error() exceptions in nanodbc")
-set(NANODBC_USE_UINT8_FOR_TINYINT ON "Enable use of uint8 to represent TINYINT fields instead of int16 in nanodbc")
+set(NANODBC_DISABLE_MSSQL_TVP ON CACHE BOOL "Disable MSSQL Table-valued parameters in nanodbc")
+set(NANODBC_DISABLE_NULL_ACCESS_ERROR ON CACHE BOOL "Disable null_access_error() exceptions in nanodbc")
+set(NANODBC_USE_UINT8_FOR_TINYINT ON CACHE BOOL "Enable use of uint8 to represent TINYINT fields instead of int16 in nanodbc")
+
+# Setup output directories
+set(CMAKE_RUNTIME_OUTPUT_DIRECTORY "${OPENKO_BIN_DIR}/${CMAKE_BUILD_TYPE}")
+
+set(OPENKO_SERVER_MAP_SRC_DIR "${CMAKE_SOURCE_DIR}/Server/bin/MAP")
+set(OPENKO_SERVER_MAP_DST_DIR "${OPENKO_BIN_DIR}/MAP")
+set(OPENKO_SERVER_QUESTS_SRC_DIR "${CMAKE_SOURCE_DIR}/Server/bin/QUESTS")
+set(OPENKO_SERVER_QUESTS_DST_DIR "${OPENKO_BIN_DIR}/QUESTS")
+
+# Multi-config generators
+foreach(CONFIG_TYPE Debug Release RelWithDebInfo MinSizeRel)
+  set(CMAKE_RUNTIME_OUTPUT_DIRECTORY_${CONFIG_TYPE} "${OPENKO_BIN_DIR}")
+endforeach()
 
 # Setup dependency dir
 set(OPENKO_DEPS_DIR ${CMAKE_SOURCE_DIR}/deps)
@@ -60,9 +76,9 @@ if(OPENKO_BUILD_SERVERS)
   fetchcontent_makeavailable(Boost)
   find_package(Boost 1.89.0 EXACT REQUIRED COMPONENTS interprocess)
 
-  add_subdirectory(deps/FTXUI)
+  add_subdirectory(deps/FTXUI EXCLUDE_FROM_ALL)
 
-  add_subdirectory(deps/nanodbc)
+  add_subdirectory(deps/nanodbc EXCLUDE_FROM_ALL)
   target_compile_features(nanodbc PUBLIC cxx_std_20)
 
   # codecvt conversions are being deprecated.
@@ -72,11 +88,20 @@ if(OPENKO_BUILD_SERVERS)
       $<$<CXX_COMPILER_ID:Clang>:-Wno-deprecated-declarations>
   )
 
-  add_subdirectory(deps/db-models-cmake)
-  add_subdirectory(deps/db-library-cmake)
+  add_subdirectory(deps/db-models-cmake EXCLUDE_FROM_ALL)
+  add_subdirectory(deps/db-library-cmake EXCLUDE_FROM_ALL)
 
   add_subdirectory(Server/shared-server)
-  
+
+  add_custom_target(copy-server-assets ALL
+    COMMENT "Copying server assets"
+    COMMAND ${CMAKE_COMMAND} -E echo "Copying server MAP dir to ${OPENKO_SERVER_MAP_DST_DIR}"
+    COMMAND ${CMAKE_COMMAND} -E copy_directory "${OPENKO_SERVER_MAP_SRC_DIR}" "${OPENKO_SERVER_MAP_DST_DIR}"
+    COMMAND ${CMAKE_COMMAND} -E echo "Copying server QUESTS dir to ${OPENKO_SERVER_QUESTS_DST_DIR}"
+    COMMAND ${CMAKE_COMMAND} -E copy_directory "${OPENKO_SERVER_QUESTS_SRC_DIR}" "${OPENKO_SERVER_QUESTS_DST_DIR}"
+    COMMAND ${CMAKE_COMMAND} -E echo "Server assets copied."
+  )
+
   ###############
   # Servers
   ###############

Server/AIServer/CMakeLists.txt

@@ -40,8 +40,11 @@ add_executable(AIServer
   ZoneEventThread.h
 )
 
+# Ensure we require the server assets to be copied
+add_dependencies(AIServer copy-server-assets)
+
 if(WIN32)
-    target_sources(AIServer PRIVATE AIServer.rc)
+  target_sources(AIServer PRIVATE AIServer.rc)
 endif()
 
 # Use C++23
@@ -68,3 +71,13 @@ target_compile_options(AIServer PRIVATE
   $<$<CXX_COMPILER_ID:GNU>:-Wall>
   $<$<CXX_COMPILER_ID:Clang>:-Wall -Wextra>
 )
+
+if(MSVC)
+  set_property(TARGET AIServer PROPERTY
+    VS_DEBUGGER_WORKING_DIRECTORY "$<TARGET_FILE_DIR:AIServer>"
+  )
+endif()
+
+add_custom_command(TARGET AIServer POST_BUILD
+  COMMAND ${CMAKE_COMMAND} -E echo "$<TARGET_FILE:AIServer> has been built"
+)

Server/AIServer/Npc.cpp

@@ -2936,7 +2936,7 @@ bool CNpc::StepNoPathMove(int /*nStep*/)
 //	NPC와 Target 과의 거리가 지정 범위보다 작은지 판단
 int CNpc::IsCloseTarget(int nRange, int Flag)
 {
-	__Vector3 vUser, vWillUser, vNpc, vDistance;
+	__Vector3 vUser = {}, vWillUser = {}, vNpc = {}, vDistance = {};
 	CUser* pUser = nullptr;
 	CNpc* pNpc = nullptr;
 	float fDis = 0.0f, fWillDis = 0.0f, fX = 0.0f, fZ = 0.0f;
@@ -3101,7 +3101,7 @@ int CNpc::GetTargetPath(int option)
 	CUser* targetUser = nullptr;
 	CNpc* npcTarget = nullptr;
 	float chaseRange = 0.0f;
-	__Vector3 vUser, vNpc, vDistance, vEnd22;
+	__Vector3 vUser = {}, vNpc = {}, vDistance = {}, vEnd22 = {};
 	float fDis = 0.0f;
 	float fDegree = 0.0f, fTargetDistance = 0.0f;
 	float fSurX = 0.0f, fSurZ = 0.0f;
@@ -3888,10 +3888,7 @@ void CNpc::MoveAttack()
 {
 	char pBuf[1024] = {};
 	int index = 0;
-	__Vector3 vUser;
-	__Vector3 vNpc;
-	__Vector3 vDistance;
-	__Vector3 vEnd22;
+	__Vector3 vUser = {}, vNpc = {}, vDistance = {}, vEnd22 = {};
 	CUser* pUser = nullptr;
 	CNpc* pNpc = nullptr;
 	float fDis = 0.0f;
@@ -4173,7 +4170,7 @@ bool CNpc::GetTargetPos(float& x, float& z)
 //	Target 과 NPC 간에 길찾기를 다시한다.
 bool CNpc::ResetPath()
 {
-	float cur_x, cur_z;
+	float cur_x = 0.0f, cur_z = 0.0f;
 	GetTargetPos(cur_x, cur_z);
 
 //	TRACE(_T("ResetPath : user pos ,, x=%.2f, z=%.2f\n"), cur_x, cur_z);

Server/Aujard/CMakeLists.txt

@@ -39,3 +39,14 @@ target_compile_options(Aujard PRIVATE
   $<$<CXX_COMPILER_ID:GNU>:-Wall>
   $<$<CXX_COMPILER_ID:Clang>:-Wall -Wextra>
 )
+
+if(MSVC)
+  # Override default working directory for ease-of-use
+  set_property(TARGET Aujard PROPERTY
+    VS_DEBUGGER_WORKING_DIRECTORY "$<TARGET_FILE_DIR:Aujard>"
+  )
+endif()
+
+add_custom_command(TARGET Aujard POST_BUILD
+  COMMAND ${CMAKE_COMMAND} -E echo "$<TARGET_FILE:Aujard> has been built"
+)

Server/Ebenezer/CMakeLists.txt

@@ -52,8 +52,11 @@ add_executable(Ebenezer
   User.h
 )
 
+# Ensure we require the server assets to be copied
+add_dependencies(AIServer copy-server-assets)
+
 if(WIN32)
-    target_sources(Ebenezer PRIVATE Ebenezer.rc)
+  target_sources(Ebenezer PRIVATE Ebenezer.rc)
 endif()
 
 # Use C++23
@@ -80,3 +83,14 @@ target_compile_options(Ebenezer PRIVATE
   $<$<CXX_COMPILER_ID:GNU>:-Wall>
   $<$<CXX_COMPILER_ID:Clang>:-Wall -Wextra>
 )
+
+if(MSVC)
+  # Override default working directory for ease-of-use
+  set_property(TARGET Ebenezer PROPERTY
+    VS_DEBUGGER_WORKING_DIRECTORY "$<TARGET_FILE_DIR:Ebenezer>"
+  )
+endif()
+
+add_custom_command(TARGET Ebenezer POST_BUILD
+  COMMAND ${CMAKE_COMMAND} -E echo "$<TARGET_FILE:Ebenezer> has been built"
+)

Server/ItemManager/CMakeLists.txt

@@ -38,3 +38,14 @@ target_compile_options(ItemManager PRIVATE
   $<$<CXX_COMPILER_ID:GNU>:-Wall>
   $<$<CXX_COMPILER_ID:Clang>:-Wall -Wextra>
 )
+
+if(MSVC)
+  # Override default working directory for ease-of-use
+  set_property(TARGET ItemManager PROPERTY
+    VS_DEBUGGER_WORKING_DIRECTORY "$<TARGET_FILE_DIR:ItemManager>"
+  )
+endif()
+
+add_custom_command(TARGET ItemManager POST_BUILD
+  COMMAND ${CMAKE_COMMAND} -E echo "$<TARGET_FILE:ItemManager> has been built"
+)

Server/VersionManager/CMakeLists.txt

@@ -39,3 +39,14 @@ target_compile_options(VersionManager PRIVATE
   $<$<CXX_COMPILER_ID:GNU>:-Wall>
   $<$<CXX_COMPILER_ID:Clang>:-Wall -Wextra>
 )
+
+if(MSVC)
+  # Override default working directory for ease-of-use
+  set_property(TARGET VersionManager PROPERTY
+    VS_DEBUGGER_WORKING_DIRECTORY "$<TARGET_FILE_DIR:VersionManager>"
+  )
+endif()
+
+add_custom_command(TARGET VersionManager POST_BUILD
+  COMMAND ${CMAKE_COMMAND} -E echo "$<TARGET_FILE:VersionManager> has been built"
+)