Disable --dependency-file; Newer versions of CMake seems to enable th…

…at automatically, but it breaks building PE files
ilmmatias · Jan 9, 2025 · 7ff1f7e · 7ff1f7e
1 parent 16b64e0
commit 7ff1f7e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/src/sdk/cmake/config.cmake b/src/sdk/cmake/config.cmake
@@ -21,9 +21,8 @@ set(CMAKE_CXX_STANDARD 23)
 
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wall -Wextra -Werror")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Werror")
+set(CMAKE_RC_FLAGS "--target=${TARGET_${ARCH}}-w64-mingw32")
 
 set(CMAKE_LINK_DEF_FILE_FLAG "")
 
-set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
-
 add_link_options(-nostdlib -nodefaultlibs)
diff --git a/src/sdk/cmake/toolchain.cmake b/src/sdk/cmake/toolchain.cmake
@@ -5,12 +5,14 @@ set(CMAKE_SYSTEM_NAME Palladium)
 set(CMAKE_SYSTEM_PROCESSOR ${ARCH})
 
 set(CMAKE_C_COMPILER clang)
-set(CMAKE_C_COMPILER_WORKS 1)
-set(CMAKE_CXX_COMPILER clang)
-set(CMAKE_CXX_COMPILER_WORKS 1)
+set(CMAKE_CXX_COMPILER clang++)
 set(CMAKE_ASM_COMPILER clang)
+set(CMAKE_RC_COMPILER llvm-windres)
 set(CMAKE_AR llvm-ar)
 
-# We're missing a message compiler for now (probably just build the one from binutils?)
-set(CMAKE_RC_COMPILER llvm-windres)
-set(CMAKE_RC_FLAGS --target=${TARGET_${ARCH}}-w64-mingw32)
+set(CMAKE_CXX_COMPILER_WORKS 1)
+set(CMAKE_C_COMPILER_WORKS 1)
+
+# Disable --dependency-file, it's not supported for .exe (PE) targets
+set(CMAKE_C_LINKER_DEPFILE_SUPPORTED FALSE)
+set(CMAKE_CXX_LINKER_DEPFILE_SUPPORTED FALSE)