cmake: make DaemonPlatform/Architecture/Compiler/BuildInfo autonomous and reusable

CMakeLists.txt

@@ -85,8 +85,7 @@ if (Daemon_OUT)
     set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${Daemon_OUT})
 endif()
 
-include(DaemonBuildInfo)
-include(DaemonPlatform)
+include(DaemonPlatform/Platform)
 
 ################################################################################
 # Configuration options

cmake/DaemonFlags.cmake

@@ -396,7 +396,7 @@ else()
 		try_c_cxx_flag(FNO_STRICT_OVERFLOW "-fno-strict-overflow")
 		try_c_cxx_flag(WSTACK_PROTECTOR "-Wstack-protector")
 
-		if (NOT NACL OR (NACL AND GAME_PIE))
+		if (NOT NACL OR (NACL AND NACL_PIE))
 			# The -pie flag requires -fPIC:
 			# > ld: error: relocation R_X86_64_64 cannot be used against local symbol; recompile with -fPIC
 			# This flag isn't used on macOS:

cmake/DaemonGame.cmake

@@ -39,8 +39,9 @@ option(BUILD_GAME_NATIVE_DLL "Build the shared library files, mostly useful for
 option(BUILD_GAME_NATIVE_EXE "Build native executable, which might be used for better performances by server owners" OFF)
 
 include(ExternalProject)
-include(DaemonBuildInfo)
-include(DaemonPlatform)
+include(DaemonPlatform/Platform)
+# TODO: Delete when Game uses NACL_PIE instead of GAME_PIE.
+set(GAME_PIE ${NACL_PIE})
 
 # Do not report unused native compiler if native vms are not built.
 # If only NACL vms are built, this will be reported in chainloaded build.

cmake/DaemonArchitecture.cmake → cmake/DaemonPlatform/Architecture.cmake

@@ -32,7 +32,7 @@
 
 try_compile(BUILD_RESULT
 	"${CMAKE_BINARY_DIR}"
-	"${DAEMON_DIR}/cmake/DaemonArchitecture/DaemonArchitecture.cpp"
+	"${CMAKE_CURRENT_LIST_DIR}/Architecture/Architecture.cpp"
 	CMAKE_FLAGS CMAKE_OSX_ARCHITECTURES=${CMAKE_OSX_ARCHITECTURES}
 	OUTPUT_VARIABLE BUILD_LOG
 )
@@ -46,7 +46,7 @@ try_compile(BUILD_RESULT
 # Setting USE_WERROR to ON doesn't print this warning.
 if (NOT BUILD_RESULT)
 	message(WARNING
-		"Failed to build DaemonArchitecture.cpp\n"
+		"Failed to build Architecture.cpp\n"
 		"Setting -Werror in CXXFLAGS can produce false positive errors\n"
 		"${BUILD_LOG}"
 	)
@@ -57,7 +57,7 @@ string(REPLACE "DAEMON_ARCH_" "" ARCH "${ARCH_DEFINE}")
 
 if (NOT ARCH)
 	message(FATAL_ERROR
-		"Missing DAEMON_ARCH, there is a mistake in DaemonArchitecture.cpp\n"
+		"Missing DAEMON_ARCH, there is a mistake in Architecture.cpp\n"
 		"${BUILD_LOG}"
 	)
 elseif(ARCH STREQUAL "unsupported")

cmake/DaemonArchitecture/DaemonArchitecture.cpp → cmake/DaemonPlatform/Architecture/Architecture.cpp

@@ -32,17 +32,6 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 platforms including ppc64, but we know how to do it for them. */
 
 #include <stdint.h>
-#include "../../src/common/Endian.h"
-
-/* qprocessordetection.h will print an error if it fails to detect
-endianness and while it is not already set, so the else clause is
-outsourced to that qprocessordetection.h file instead. */
-
-#if defined(Q3_BIG_ENDIAN)
-	#define Q_BYTE_ORDER Q_BIG_ENDIAN
-#elif defined(Q3_LITTLE_ENDIAN)
-	#define Q_BYTE_ORDER Q_LITTLE_ENDIAN
-#endif
 
 /* This source file includes qprocessordetection.h from Qt:
 

cmake/DaemonCompiler.cmake → cmake/DaemonPlatform/Compiler.cmake

@@ -39,7 +39,7 @@ function(detect_daemon_compiler lang)
 
 	try_compile(BUILD_RESULT
 		"${CMAKE_BINARY_DIR}"
-		"${DAEMON_DIR}/cmake/DaemonCompiler/DaemonCompiler${${lang}_EXT}"
+		"${CMAKE_CURRENT_LIST_DIR}/Compiler/Compiler${${lang}_EXT}"
 		CMAKE_FLAGS CMAKE_OSX_ARCHITECTURES=${CMAKE_OSX_ARCHITECTURES}
 		OUTPUT_VARIABLE BUILD_LOG
 	)

cmake/DaemonPlatform/Compiler/Compiler.cpp

@@ -0,0 +1 @@
+#include "Compiler.c"

cmake/DaemonCompiler/DaemonCompiler.sh → cmake/DaemonPlatform/Compiler/Compiler.sh

@@ -32,7 +32,7 @@
 set -ueo pipefail
 
 script_dir="$(cd "$(dirname "${BASH_SOURCE[0]}")" >/dev/null 2>&1 && pwd)"
-file_path="${script_dir}/DaemonCompiler.c"
+file_path="${script_dir}/Compiler.c"
 
 # PNaCl doesn't work with “-o /dev/null” as it uses the output path as a
 # pattern for temporary files and then the parent folder should be writable.

cmake/DaemonPlatform/Platform.cmake

@@ -0,0 +1,12 @@
+include(${CMAKE_CURRENT_LIST_DIR}/BuildInfo.cmake)
+include(${CMAKE_CURRENT_LIST_DIR}/System.cmake)
+include(${CMAKE_CURRENT_LIST_DIR}/Architecture.cmake)
+include(${CMAKE_CURRENT_LIST_DIR}/Compiler.cmake)
+
+if (NACL AND DAEMON_CXX_COMPILER_Saigo)
+	# Saigo clang reports weird errors when building some Unvanquished cgame and sgame arm nexe with PIE.
+	# Saigo clang crashes when building Unvanquished amd64 cgame with PIE, sgame builds properly though.
+	set(NACL_PIE 0)
+else()
+	set(NACL_PIE 1)
+endif()

cmake/DaemonPlatform.cmake → cmake/DaemonPlatform/System.cmake

@@ -39,15 +39,3 @@ elseif( NACL )
 else()
   message( FATAL_ERROR "Platform not supported" )
 endif()
-
-if (NACL AND USE_NACL_SAIGO)
-	# Saigo clang reports weird errors when building some cgame and sgame arm nexe with PIE.
-	# Saigo clang crashes when building amd64 cgame with PIE, sgame builds properly though.
-	set(GAME_PIE 0)
-else()
-	set(GAME_PIE 1)
-endif()
-
-
-include(DaemonArchitecture)
-include(DaemonCompiler)