From f70c45782ea15dadee29efd2a72e56a1099a2efc Mon Sep 17 00:00:00 2001 From: bjarni Date: Thu, 30 Mar 2006 20:14:06 +0000 Subject: [PATCH] (svn r4183) -Codechange: [Makefile]: removed MANUAL_CONFIG as it's not used anymore This should hopefully fix the issue where WITH_SDL can be defined while SDL-CONFIG is not Added an error if WITH_SDL is defined but SDL-CONFIG is not. The same goes for WITH_PNG even though nobody reported this as a problem removed STATIC_ZLIB_PATH as it turns out that nobody used it --- Makefile | 67 ++++++------------------------ makefiledir/Makefile.config_writer | 1 - makefiledir/Makefile.libdetection | 3 ++ 3 files changed, 15 insertions(+), 56 deletions(-) diff --git a/Makefile b/Makefile index 4de699a2ac..f48ad707a1 100644 --- a/Makefile +++ b/Makefile @@ -46,7 +46,6 @@ # WITH_COCOA: Cocoa video driver support # # Summary of other defines: -# MANUAL_CONFIG: do not use Makefile.config, config options set manually # DEBUG: build in debug mode # PROFILE: build in profile mode, disables -s and -fomit-frame-pointer # TRANSLATOR: build in translator mode (untranslated strings are prepended by @@ -134,10 +133,6 @@ # it checks if the version tag in Makefile.config is the same and force update outdated config files MAKEFILE_VERSION:=10 -# CONFIG_WRITER has to be found even for manual configuration -CONFIG_WRITER=makefiledir/Makefile.config_writer - -ifndef MANUAL_CONFIG # Automatic configuration MAKE_CONFIG:=Makefile.config MAKEFILE:=Makefile @@ -148,30 +143,10 @@ CONFIG_WRITER=makefiledir/Makefile.config_writer # See target section for how this is built, suppress errors # since first time it isn't found but make reads this twice -include $(MAKE_CONFIG) -else -CONFIG_INCLUDED:=1 -endif -ifndef LIBPNG-CONFIG -LIBPNG-CONFIG :=libpng-config -endif # updates Makefile.config if it's outdated ifneq ($(MAKEFILE_VERSION),$(CONFIG_VERSION)) - ifndef MANUAL_CONFIG # manual config should not check this - UPDATECONFIG:=upgradeconf - CONFIG_INCLUDED:= - else - # this should define SDL-CONFIG for manual configuration - ifeq ($(shell uname),FreeBSD) - SDL-CONFIG:=sdl11-config - else - SDL-CONFIG:=sdl-config - endif - endif -endif - -ifndef SDL-CONFIG UPDATECONFIG:=upgradeconf CONFIG_INCLUDED:= endif @@ -248,6 +223,17 @@ ifdef OSX LDFLAGS+=-framework Cocoa endif +ifdef WITH_SDL + ifndef SDL-CONFIG +$(error WITH_SDL can't be used when SDL-CONFIG is not set. Edit Makefile.config to correct this) + endif +endif + +ifdef WITH_PNG + ifndef LIBPNG-CONFIG +$(error WITH_PNG can't be used when LIBPNG-CONFIG is not set. Edit Makefile.config to correct this) + endif +endif ############################################################################## # @@ -431,36 +417,7 @@ endif # zlib config ifdef WITH_ZLIB CDEFS += -DWITH_ZLIB - ifdef STATIC - ifdef OSX -# zlib is default on OSX, so everybody have it. No need for static linking - LIBS += -lz - else - ifndef STATIC_ZLIB_PATH - ifndef MANUAL_CONFIG - # updates Makefile.config with the zlib path - UPDATECONFIG:=upgradeconf - endif - TEMP:=$(shell ls /lib 2>/dev/null | grep "zlib.a")$(shell ls /lib 2>/dev/null | grep "libz.a") - ifdef TEMP - STATIC_ZLIB_PATH:=/lib/$(TEMP) - else - TEMP:=$(shell ls /usr/lib 2>/dev/null | grep "zlib.a")$(shell ls /usr/lib 2>/dev/null | grep "libz.a") - ifdef TEMP - STATIC_ZLIB_PATH:=/usr/lib/$(TEMP) - else - TEMP:=$(shell ls /usr/local/lib 2>/dev/null | grep "zlib.a")$(shell ls /usr/local/lib 2>/dev/null | grep "libz.a") - ifdef TEMP - STATIC_ZLIB_PATH:=/usr/local/lib/$(TEMP) - endif - endif - endif - endif - LIBS += $(STATIC_ZLIB_PATH) - endif - else - LIBS += -lz - endif + LIBS += -lz endif # libpng config diff --git a/makefiledir/Makefile.config_writer b/makefiledir/Makefile.config_writer index a305c89d0b..2f3351a32d 100644 --- a/makefiledir/Makefile.config_writer +++ b/makefiledir/Makefile.config_writer @@ -67,7 +67,6 @@ $(MAKE_CONFIG): $(call CONFIG_LINE,WITH_ZLIB:=$(WITH_ZLIB)) $(call CONFIG_LINE,WITH_SDL:=$(WITH_SDL)) $(call CONFIG_LINE,WITH_PNG:=$(WITH_PNG)) - $(call CONFIG_LINE,STATIC_ZLIB_PATH:=$(STATIC_ZLIB_PATH)) $(call CONFIG_LINE,WITH_COCOA:=$(WITH_COCOA)) $(call CONFIG_LINE,) diff --git a/makefiledir/Makefile.libdetection b/makefiledir/Makefile.libdetection index 237c02f98a..a8859505f1 100644 --- a/makefiledir/Makefile.libdetection +++ b/makefiledir/Makefile.libdetection @@ -68,6 +68,9 @@ else SDL-CONFIG:=sdl-config endif +# set libpng-config to the default value +LIBPNG-CONFIG :=libpng-config + # Networking, enabled by default WITH_NETWORK:=1