Browse Source

Remove unnecessary distinction between 32bit and 64bit on osx

Palana 11 years ago
parent
commit
c1599a069a
4 changed files with 43 additions and 20 deletions
  1. 39 16
      cmake/Modules/ObsHelpers.cmake
  2. 1 1
      cmake/osxbundle/obslaunch.sh
  3. 2 2
      libobs/obs-cocoa.c
  4. 1 1
      obs/platform-osx.mm

+ 39 - 16
cmake/Modules/ObsHelpers.cmake

@@ -20,17 +20,30 @@ if(INSTALLER_RUN AND NOT DEFINED ENV{obsInstallerTempDir})
 endif()
 endif()
 
 
 if(NOT UNIX_STRUCTURE)
 if(NOT UNIX_STRUCTURE)
-	set(OBS_EXECUTABLE_DESTINATION "bin/${_lib_suffix}bit")
-	set(OBS_EXECUTABLE32_DESTINATION "bin/32bit")
-	set(OBS_EXECUTABLE64_DESTINATION "bin/64bit")
-	set(OBS_LIBRARY_DESTINATION "bin/${_lib_suffix}bit")
-	set(OBS_LIBRARY32_DESTINATION "bin/32bit")
-	set(OBS_LIBRARY64_DESTINATION "bin/64bit")
-	set(OBS_PLUGIN_DESTINATION "obs-plugins/${_lib_suffix}bit")
-	set(OBS_PLUGIN32_DESTINATION "obs-plugins/32bit")
-	set(OBS_PLUGIN64_DESTINATION "obs-plugins/64bit")
 	set(OBS_DATA_DESTINATION "data")
 	set(OBS_DATA_DESTINATION "data")
-	add_definitions(-DOBS_DATA_PATH="../../${OBS_DATA_DESTINATION}")
+	if(APPLE)
+		set(OBS_EXECUTABLE_DESTINATION "bin")
+		set(OBS_EXECUTABLE32_DESTINATION "bin")
+		set(OBS_EXECUTABLE64_DESTINATION "bin")
+		set(OBS_LIBRARY_DESTINATION "bin")
+		set(OBS_LIBRARY32_DESTINATION "bin")
+		set(OBS_LIBRARY64_DESTINATION "bin")
+		set(OBS_PLUGIN_DESTINATION "obs-plugins")
+		set(OBS_PLUGIN32_DESTINATION "obs-plugins")
+		set(OBS_PLUGIN64_DESTINATION "obs-plugins")
+		add_definitions(-DOBS_DATA_PATH="../${OBS_DATA_DESTINATION}")
+	else()
+		set(OBS_EXECUTABLE_DESTINATION "bin/${_lib_suffix}bit")
+		set(OBS_EXECUTABLE32_DESTINATION "bin/32bit")
+		set(OBS_EXECUTABLE64_DESTINATION "bin/64bit")
+		set(OBS_LIBRARY_DESTINATION "bin/${_lib_suffix}bit")
+		set(OBS_LIBRARY32_DESTINATION "bin/32bit")
+		set(OBS_LIBRARY64_DESTINATION "bin/64bit")
+		set(OBS_PLUGIN_DESTINATION "obs-plugins/${_lib_suffix}bit")
+		set(OBS_PLUGIN32_DESTINATION "obs-plugins/32bit")
+		set(OBS_PLUGIN64_DESTINATION "obs-plugins/64bit")
+		add_definitions(-DOBS_DATA_PATH="../../${OBS_DATA_DESTINATION}")
+	endif()
 else()
 else()
 	set(OBS_EXECUTABLE_DESTINATION "bin")
 	set(OBS_EXECUTABLE_DESTINATION "bin")
 	set(OBS_EXECUTABLE32_DESTINATION "bin32")
 	set(OBS_EXECUTABLE32_DESTINATION "bin32")
@@ -104,9 +117,13 @@ endfunction()
 
 
 macro(install_obs_core target)
 macro(install_obs_core target)
 	if(CMAKE_SIZEOF_VOID_P EQUAL 8)
 	if(CMAKE_SIZEOF_VOID_P EQUAL 8)
-		set(_lib_suffix 64)
+		set(_bit_suffix "64bit/")
 	else()
 	else()
-		set(_lib_suffix 32)
+		set(_bit_suffix "32bit/")
+	endif()
+
+	if(APPLE)
+		set(_bit_suffix "")
 	endif()
 	endif()
 
 
 	install(TARGETS ${target}
 	install(TARGETS ${target}
@@ -114,7 +131,8 @@ macro(install_obs_core target)
 		RUNTIME DESTINATION "${OBS_EXECUTABLE_DESTINATION}")
 		RUNTIME DESTINATION "${OBS_EXECUTABLE_DESTINATION}")
 	add_custom_command(TARGET ${target} POST_BUILD
 	add_custom_command(TARGET ${target} POST_BUILD
 		COMMAND "${CMAKE_COMMAND}" -E copy
 		COMMAND "${CMAKE_COMMAND}" -E copy
-			"$<TARGET_FILE:${target}>" "${OBS_OUTPUT_DIR}/$<CONFIGURATION>/bin/${_lib_suffix}bit/$<TARGET_FILE_NAME:${target}>"
+			"$<TARGET_FILE:${target}>"
+			"${OBS_OUTPUT_DIR}/$<CONFIGURATION>/bin/${_bit_suffix}$<TARGET_FILE_NAME:${target}>"
 		VERBATIM)
 		VERBATIM)
 
 
 	if(DEFINED ENV{obsInstallerTempDir})
 	if(DEFINED ENV{obsInstallerTempDir})
@@ -135,9 +153,13 @@ endmacro()
 
 
 macro(install_obs_plugin target)
 macro(install_obs_plugin target)
 	if(CMAKE_SIZEOF_VOID_P EQUAL 8)
 	if(CMAKE_SIZEOF_VOID_P EQUAL 8)
-		set(_lib_suffix 64)
+		set(_bit_suffix "64bit/")
 	else()
 	else()
-		set(_lib_suffix 32)
+		set(_bit_suffix "32bit/")
+	endif()
+
+	if(APPLE)
+		set(_bit_suffix "")
 	endif()
 	endif()
 
 
 	install(TARGETS ${target}
 	install(TARGETS ${target}
@@ -145,7 +167,8 @@ macro(install_obs_plugin target)
 		RUNTIME DESTINATION "${OBS_PLUGIN_DESTINATION}")
 		RUNTIME DESTINATION "${OBS_PLUGIN_DESTINATION}")
 	add_custom_command(TARGET ${target} POST_BUILD
 	add_custom_command(TARGET ${target} POST_BUILD
 		COMMAND "${CMAKE_COMMAND}" -E copy
 		COMMAND "${CMAKE_COMMAND}" -E copy
-			"$<TARGET_FILE:${target}>" "${OBS_OUTPUT_DIR}/$<CONFIGURATION>/obs-plugins/${_lib_suffix}bit/$<TARGET_FILE_NAME:${target}>"
+			"$<TARGET_FILE:${target}>"
+			"${OBS_OUTPUT_DIR}/$<CONFIGURATION>/obs-plugins/${_bit_suffix}$<TARGET_FILE_NAME:${target}>"
 		VERBATIM)
 		VERBATIM)
 
 
 	if(DEFINED ENV{obsInstallerTempDir})
 	if(DEFINED ENV{obsInstallerTempDir})

+ 1 - 1
cmake/osxbundle/obslaunch.sh

@@ -1,5 +1,5 @@
 #!/bin/sh
 #!/bin/sh
 cd "$(dirname "$0")"
 cd "$(dirname "$0")"
-cd ../Resources
+cd ../Resources/bin
 exec ./obs "$@"
 exec ./obs "$@"
 
 

+ 2 - 2
libobs/obs-cocoa.c

@@ -26,8 +26,8 @@
 static const char *plugin_patterns[] = {
 static const char *plugin_patterns[] = {
 	OBS_INSTALL_PREFIX "obs-plugins/%s.so",
 	OBS_INSTALL_PREFIX "obs-plugins/%s.so",
 	OBS_INSTALL_PREFIX "obs-plugins/lib%s.so",
 	OBS_INSTALL_PREFIX "obs-plugins/lib%s.so",
-	"../../obs-plugins/64bit/%s.so",
-	"../../obs-plugins/64bit/lib%s.so"
+	"../obs-plugins/64bit/%s.so",
+	"../obs-plugins/64bit/lib%s.so"
 };
 };
 
 
 static const int plugin_patterns_size =
 static const int plugin_patterns_size =

+ 1 - 1
obs/platform-osx.mm

@@ -28,7 +28,7 @@ using namespace std;
 bool GetDataFilePath(const char *data, string &output)
 bool GetDataFilePath(const char *data, string &output)
 {
 {
 	stringstream str;
 	stringstream str;
-	str << OBS_DATA_PATH << "/obs-studio/" << data;
+	str << OBS_DATA_PATH "/obs-studio/" << data;
 	output = str.str();
 	output = str.str();
 	return !access(output.c_str(), R_OK);
 	return !access(output.c_str(), R_OK);
 }
 }