Fixed inefficiencies in building the SSL bundle - in particular the constant rerunning of config.status

git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/base/trunk@30181 72102866-910b-0410-8b05-ffd578937521

ChangeLog

@@ -1,3 +1,10 @@
+2010-04-18   Nicola Pero <nicola.pero@meta-innovation.com>
+
+	Fixed inefficiencies in building the SSL bundle.
+	* SSL/Makefile.postable: Do not create GNUSTEP_TARGET_DIR/config.h
+	from config.h; do not depend on config.status.
+	* SSL/Makefile.preamble: Corresponding updates to includes.
+
 2010-04-18   Nicola Pero <nicola.pero@meta-innovation.com>
 
 	Fixed bug 25356 ("make ignoring installation domain configuration

SSL/Makefile.postamble

@@ -67,7 +67,7 @@
 
 # Things to do after distcleaning
 after-distclean::
-	rm -rf $(GNUSTEP_TARGET_DIR)/config.h config.status config.log config.cache TAGS config.mak config.h
+	rm -rf config.status config.log config.cache TAGS config.mak config.h
 
 # Things to do before checking
 # before-check::
@@ -75,21 +75,20 @@ after-distclean::
 # Things to do after checking
 # after-check::
 
-config.mak: config.mak.in config.status
+config.mak: config.mak.in
-	./config.status
-	-touch config.mak
-
-#
-# The config.h file is specific to a target
-#
-$(GNUSTEP_TARGET_DIR)/config.h: config.status
-	./config.status
-	-touch config.h
-ifneq ($(GNUSTEP_TARGET_DIR), .)
-	$(MKDIRS) $(GNUSTEP_TARGET_DIR)
-	-mv config.h $(GNUSTEP_TARGET_DIR)
-endif
-
-# Run configure manually if you need to pass any options
-config.status:
 	./configure
+
+config.h: config.h.in
+	./configure
+
+# PS: the config.h file is specific to the target that was configured.
+# At the moment, we use a single config.h file in ./, so if you want
+# to change target, you have to reconfigure the SSL bundle, which
+# would remove the configuration for the previous target and create a
+# configuration for the new target.
+#
+# A possible improvement would be to have configure store each
+# config.h file in a target-specific directory so that when you
+# reconfigure for a different target, you don't lose the configuration
+# for the previous target.  It doesn't seem to help much though as you
+# still have to configure and build multiple times.

SSL/Makefile.preamble

@@ -52,7 +52,7 @@ ADDITIONAL_CFLAGS =
 # include config.h from gnustep-base instead of the 
 # one in this directory.  It is required to find
 # GSPrivate.h on non-flattened though.
-ADDITIONAL_INCLUDE_DIRS = -I./$(GNUSTEP_TARGET_DIR) \
+ADDITIONAL_INCLUDE_DIRS = -I. \
  -I../Source/$(GNUSTEP_TARGET_DIR) -I../Source \
  -I../Headers/Additions -I../Headers