Fixes for GNUmakefile on Mac:
authorBrecht Van Lommel <brechtvanlommel@pandora.be>
Mon, 14 Mar 2011 15:40:07 +0000 (15:40 +0000)
committerBrecht Van Lommel <brechtvanlommel@pandora.be>
Mon, 14 Mar 2011 15:40:07 +0000 (15:40 +0000)
* Don't use "make install", doesn't work yet.
* Use total number of cores rather than number of processors.
* Quicker number of cores query, by not asking for all system info.

GNUmakefile
source/creator/CMakeLists.txt

index 265826f..e857106 100644 (file)
@@ -48,11 +48,14 @@ endif
 
 # 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 system_profiler | awk '/Number Of CPUs/{print $4}{next;}')
+       NPROCS:=$(shell system_profiler SPHardwareDataType | awk '/Total Number Of Cores/ {print $5}{next;};')
+       # make install not support on mac yet 
+       DEFAULT_TARGET=
 endif
 ifeq ($(OS), FreeBSD)
        NPROCS:=$(shell sysctl -a | grep "hw.ncpu " | cut -d" " -f3 )
@@ -75,7 +78,7 @@ all:
 
        @echo 
        @echo Building Blender ...
-       cd $(BUILD_DIR) ; make -s -j $(NPROCS) install
+       cd $(BUILD_DIR) ; make -s -j $(NPROCS) ${DEFAULT_TARGET} 
        @echo 
        @echo run blender from "$(BUILD_DIR)/bin/blender"
        @echo 
index 436d2f4..561f28a 100644 (file)
@@ -529,6 +529,7 @@ if(WITH_INSTALL)
 
        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)