[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [35575] trunk/blender: Mac OS X cmake changes and bundle fixes

Brecht Van Lommel brechtvanlommel at pandora.be
Wed Mar 16 16:44:18 CET 2011


Revision: 35575
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=35575
Author:   blendix
Date:     2011-03-16 15:44:17 +0000 (Wed, 16 Mar 2011)
Log Message:
-----------
Mac OS X cmake changes and bundle fixes

For cmake users, you should now use "make install" instead of "make". This was
already changed for Linux and Windows. Same for Xcode, use the install target.

Changes:
* CMake install mechanism, resulting bundle was verified to be indentical here.
* For cmake, include Info.plist in bundle using builtin mechanism for that.
  There was some code in packaging.cmake, but it wasn't correct as these
  properties need to be set on the executable.
* For scons, fix app bundle version, was still using removed release/VERSION.
* Remove unused blendercreator.app and blenderpublisher.app.
* Fix Info.plist being set as a binary file in svn, should be plain text.

Modified Paths:
--------------
    trunk/blender/GNUmakefile
    trunk/blender/build_files/cmake/packaging.cmake
    trunk/blender/build_files/scons/tools/Blender.py
    trunk/blender/source/creator/CMakeLists.txt
    trunk/blender/source/darwin/blender.app/Contents/Info.plist
    trunk/blender/source/darwin/blenderplayer.app/Contents/Info.plist

Removed Paths:
-------------
    trunk/blender/source/darwin/blendercreator.app/
    trunk/blender/source/darwin/blenderpublisher.app/

Property Changed:
----------------
    trunk/blender/source/darwin/blender.app/Contents/Info.plist
    trunk/blender/source/darwin/blenderplayer.app/Contents/Info.plist

Modified: trunk/blender/GNUmakefile
===================================================================
--- trunk/blender/GNUmakefile	2011-03-16 14:21:35 UTC (rev 35574)
+++ trunk/blender/GNUmakefile	2011-03-16 15:44:17 UTC (rev 35575)
@@ -48,14 +48,11 @@
 
 # Get the number of cores for threaded build
 NPROCS:=1
-DEFAULT_TARGET=install
 ifeq ($(OS), Linux)
 	NPROCS:=$(shell grep -c ^processor /proc/cpuinfo)
 endif
 ifeq ($(OS), Darwin)
 	NPROCS:=$(shell sysctl -a | grep "hw.ncpu " | cut -d" " -f3)
-	# make install not support on mac yet 
-	DEFAULT_TARGET=
 endif
 ifeq ($(OS), FreeBSD)
 	NPROCS:=$(shell sysctl -a | grep "hw.ncpu " | cut -d" " -f3 )
@@ -78,7 +75,7 @@
 
 	@echo 
 	@echo Building Blender ...
-	cd $(BUILD_DIR) ; make -s -j $(NPROCS) ${DEFAULT_TARGET} 
+	cd $(BUILD_DIR) ; make -s -j $(NPROCS) install
 	@echo 
 	@echo run blender from "$(BUILD_DIR)/bin/blender"
 	@echo 

Modified: trunk/blender/build_files/cmake/packaging.cmake
===================================================================
--- trunk/blender/build_files/cmake/packaging.cmake	2011-03-16 14:21:35 UTC (rev 35574)
+++ trunk/blender/build_files/cmake/packaging.cmake	2011-03-16 15:44:17 UTC (rev 35575)
@@ -51,12 +51,6 @@
 
 	# Libraries are bundled directly
 	set(CPACK_COMPONENT_LIBRARIES_HIDDEN TRUE)
-
-	# Bundle Properties
-	set(MACOSX_BUNDLE_BUNDLE_NAME blender)
-	set(MACOSX_BUNDLE_BUNDLE_VERSION ${BLENDER_VERSION})
-	set(MACOSX_BUNDLE_SHORT_VERSION_STRING ${BLENDER_VERSION})
-	set(MACOSX_BUNDLE_LONG_VERSION_STRING "Version ${BLENDER_VERSION}-r${BUILD_REV}")
 endif(APPLE)
 
 set(CPACK_PACKAGE_EXECUTABLES "blender")

Modified: trunk/blender/build_files/scons/tools/Blender.py
===================================================================
--- trunk/blender/build_files/scons/tools/Blender.py	2011-03-16 14:21:35 UTC (rev 35574)
+++ trunk/blender/build_files/scons/tools/Blender.py	2011-03-16 15:44:17 UTC (rev 35575)
@@ -488,7 +488,8 @@
     if os.path.isdir(cmd):
         shutil.rmtree(cmd)
     shutil.copytree(sourcedir, cmd)
-    cmd = "cat %s | sed s/VERSION/`cat release/VERSION`/ | sed s/DATE/`date +'%%Y-%%b-%%d'`/ > %s"%(sourceinfo,targetinfo)
+    cmd = "cat %s | sed s/\$\{MACOSX_BUNDLE_SHORT_VERSION_STRING\}/%s/ | "%(sourceinfo,VERSION)
+    cmd += "sed s/\$\{MACOSX_BUNDLE_LONG_VERSION_STRING\}/%s,\ `date +'%%Y-%%b-%%d'`/ > %s"%(VERSION,targetinfo)
     commands.getoutput(cmd)
     cmd = 'cp %s/%s %s/%s.app/Contents/MacOS/%s'%(builddir, binary,installdir, binary, binary)
     commands.getoutput(cmd)

Modified: trunk/blender/source/creator/CMakeLists.txt
===================================================================
--- trunk/blender/source/creator/CMakeLists.txt	2011-03-16 14:21:35 UTC (rev 35574)
+++ trunk/blender/source/creator/CMakeLists.txt	2011-03-16 15:44:17 UTC (rev 35575)
@@ -186,7 +186,7 @@
 	# ${CMAKE_CFG_INTDIR} should replace \${BUILD_TYPE} when using add_command
 	set(TARGETDIR ${EXECUTABLE_OUTPUT_PATH}/\${BUILD_TYPE})
 elseif(APPLE)
-	set(TARGETDIR ${EXECUTABLE_OUTPUT_PATH}/${CMAKE_CFG_INTDIR})
+	set(TARGETDIR ${EXECUTABLE_OUTPUT_PATH}/\${BUILD_TYPE})
 else()
 	set(TARGETDIR ${EXECUTABLE_OUTPUT_PATH})
 endif()
@@ -528,56 +528,113 @@
 		endif()
 
 	elseif(APPLE)
-		# TODO, APPLE needs a 'make install' target like win32 and unix
-		# also update GNUmakefile to run make install if you do this
-
 		set(SOURCEDIR ${CMAKE_SOURCE_DIR}/source/darwin/blender.app)
 		set(SOURCEINFO ${SOURCEDIR}/Contents/Info.plist)
-		set(TARGETINFO ${TARGETDIR}/blender.app/Contents/Info.plist)
+		set(TARGETDIR_VER ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION})
 
-		add_custom_command(
-			TARGET blender POST_BUILD MAIN_DEPENDENCY blender
-			COMMAND cp -R ${CMAKE_SOURCE_DIR}/release/text/* ${TARGETDIR}/
+		# setup Info.plist
+		execute_process(COMMAND date "+%Y-%m-%d" OUTPUT_VARIABLE BLENDER_DATE OUTPUT_STRIP_TRAILING_WHITESPACE)
+
+		set_target_properties(blender PROPERTIES
+			MACOSX_BUNDLE_INFO_PLIST ${SOURCEINFO}
+			MACOSX_BUNDLE_SHORT_VERSION_STRING ${BLENDER_VERSION}
+			MACOSX_BUNDLE_LONG_VERSION_STRING "${BLENDER_VERSION} ${BLENDER_DATE}")
+
+		# important to make a clean  install each time else old scripts get loaded.
+		install(
+			CODE
+			"file(REMOVE_RECURSE ${TARGETDIR_VER})"
 		)
 
+		# message after building.
 		add_custom_command(
 			TARGET blender POST_BUILD MAIN_DEPENDENCY blender
-			COMMAND cp -Rf ${SOURCEINFO} ${TARGETDIR}/blender.app/Contents/
-			COMMAND cp -Rf ${SOURCEDIR}/Contents/PkgInfo ${TARGETDIR}/blender.app/Contents/
-			COMMAND cp -Rf ${SOURCEDIR}/Contents/Resources ${TARGETDIR}/blender.app/Contents/
-			COMMAND cat ${SOURCEINFO} | sed s/VERSION/${BLENDER_VERSION}${BLENDER_VERSION_CHAR}/ | sed s/DATE/`date +'%Y-%b-%d'`/ > ${TARGETINFO}
-			COMMAND rm -Rf ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}
-			COMMAND mkdir ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}
-			COMMAND mkdir ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/datafiles/
-			COMMAND cp ${CMAKE_SOURCE_DIR}/release/bin/.blender/.bfont.ttf ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/datafiles/
+			COMMAND ${CMAKE_COMMAND} -E echo 'now run: \"make install\" to copy runtime files & scripts to ${TARGETDIR_VER}'
 		)
 
+		# handy install macro to exclude files, we use \$ escape for the "to"
+		# argument when calling so ${BUILD_TYPE} does not get expanded
+		macro(install_dir from to)
+			install(
+				DIRECTORY ${from}
+				DESTINATION ${to}
+				PATTERN ".svn" EXCLUDE
+				PATTERN "*.pyc" EXCLUDE
+				PATTERN "*.pyo" EXCLUDE
+				PATTERN "*.orig" EXCLUDE
+				PATTERN "*.rej" EXCLUDE
+				PATTERN "__pycache__" EXCLUDE
+				PATTERN "__MACOSX" EXCLUDE
+				PATTERN ".DS_Store" EXCLUDE
+			)
+		endmacro()
+
+		# install release and app files
+		install_dir(
+			${CMAKE_SOURCE_DIR}/release/text/
+			\${TARGETDIR}
+		)
+
+		install(
+			FILES ${SOURCEDIR}/Contents/PkgInfo
+			DESTINATION ${TARGETDIR}/blender.app/Contents/
+		)
+
+		install_dir(
+			${SOURCEDIR}/Contents/Resources
+			\${TARGETDIR}/blender.app/Contents/
+		)
+
+		install(
+			FILES ${CMAKE_SOURCE_DIR}/release/bin/.blender/.bfont.ttf
+			DESTINATION ${TARGETDIR_VER}/datafiles
+		)
+
+		# localization
 		if(WITH_INTERNATIONAL)
-			add_custom_command(
-				TARGET blender POST_BUILD MAIN_DEPENDENCY blender
-				COMMAND cp ${CMAKE_SOURCE_DIR}/release/bin/.blender/.Blanguages ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/datafiles/
-				COMMAND cp -Rf ${CMAKE_SOURCE_DIR}/release/bin/.blender/locale ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/datafiles/
+			install(
+				FILES ${CMAKE_SOURCE_DIR}/release/bin/.blender/.Blanguages
+				DESTINATION ${TARGETDIR_VER}/datafiles
 			)
+
+			install_dir(
+				${CMAKE_SOURCE_DIR}/release/bin/.blender/locale/
+				\${TARGETDIR_VER}/datafiles/locale
+			)
 		endif()
 
+		# python
 		if(WITH_PYTHON)
+			# the python zip is first extract as part of the build process,
+			# and then later installed as part of make install. this is much
+			# quicker, and means we can easily exclude files on copy
+			add_custom_target(
+				extractpyzip
+				DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/python)
+
 			set(PYTHON_ZIP "python_${CMAKE_OSX_ARCHITECTURES}.zip")
+				
 			add_custom_command(
-				TARGET blender POST_BUILD MAIN_DEPENDENCY blender
-				COMMAND cp -Rf ${CMAKE_SOURCE_DIR}/release/scripts ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/
-				COMMAND mkdir ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/python/
-				COMMAND unzip -q ${LIBDIR}/release/${PYTHON_ZIP} -d ${TARGETDIR}/blender.app/Contents/MacOS/${BLENDER_VERSION}/python/
-				COMMAND find ${TARGETDIR}/blender.app -name '*.py[co]' -prune -exec rm -rf {} '\;'
-				COMMAND find ${TARGETDIR}/blender.app -name '__pycache__' -exec rmdir {} '+'
+				OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/python
+				COMMAND rm -rf ${CMAKE_CURRENT_BINARY_DIR}/python/
+				COMMAND mkdir ${CMAKE_CURRENT_BINARY_DIR}/python/
+				COMMAND unzip -q ${LIBDIR}/release/${PYTHON_ZIP} -d ${CMAKE_CURRENT_BINARY_DIR}/python/
+				DEPENDS ${LIBDIR}/release/${PYTHON_ZIP})
+			
+			add_dependencies(blender extractpyzip)
+
+			# copy extracted python files
+			install_dir(
+				${CMAKE_CURRENT_BINARY_DIR}/python
+				\${TARGETDIR_VER}
 			)
+
+			# copy scripts
+			install_dir(
+				${CMAKE_SOURCE_DIR}/release/scripts
+				\${TARGETDIR_VER}
+			)
 		endif()
-
-		add_custom_command(
-			TARGET blender POST_BUILD MAIN_DEPENDENCY blender
-			COMMAND find ${TARGETDIR}/blender.app -name .DS_Store -prune -exec rm -rf {} "\;"
-			COMMAND find ${TARGETDIR}/blender.app -name .svn -prune -exec rm -rf {} "\;"
-			COMMAND find ${TARGETDIR}/blender.app -name __MACOSX -prune -exec rm -rf {} "\;"
-		)
 	endif()
 
 endif()

Modified: trunk/blender/source/darwin/blender.app/Contents/Info.plist
===================================================================
(Binary files differ)


Property changes on: trunk/blender/source/darwin/blender.app/Contents/Info.plist
___________________________________________________________________
Modified: svn:mime-type
   - application/octet-stream
   + text/plain

Modified: trunk/blender/source/darwin/blenderplayer.app/Contents/Info.plist
===================================================================
(Binary files differ)


Property changes on: trunk/blender/source/darwin/blenderplayer.app/Contents/Info.plist
___________________________________________________________________
Modified: svn:mime-type
   - application/octet-stream
   + text/plain




More information about the Bf-blender-cvs mailing list