From: Phil Pennock <pdp@spodhuis.org>
Date: Sun, 16 Jan 2011 23:36:55 +0000 (-0500)
Subject: Bug-fix for bash and no-dynamic case.
X-Git-Tag: exim-4_74_RC1~11
X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/eec525c43adade97ff94d839810faf7cb35bd87f

Bug-fix for bash and no-dynamic case.

When writing the patch, originally nothing other than a cp was needed if
there were no dynamic modules.  That changed, but the guard at the top
did not.  Remove that check.

bash does not like a block which consists solely of a comment.  Provide
a ':' invocation.

Both problems spotted by Simon Arlott -- thanks.
---

diff --git a/src/scripts/lookups-Makefile b/src/scripts/lookups-Makefile
index 7069cfb69..31820ea2f 100755
--- a/src/scripts/lookups-Makefile
+++ b/src/scripts/lookups-Makefile
@@ -8,18 +8,14 @@ defs_source=Makefile
 tag_marker='MAGIC-TAG-MODS-OBJ-RULES-GO-HERE'
 
 tab='	'
-if grep -q "^LOOKUP.*=[ $tab]*2" "$defs_source"
-then
-  # we have work to do
-else
-  echo "No dynamic module loading support"
-  cp "$input" "$target"
-  exit 0
-fi
+
+# We always do something now, since there should always be a lookup,
+# and now we need to run in order to put the OBJ+= rules in.
 
 if grep -q "^CFLAGS_DYNAMIC[ $tab]*=" "$defs_source"
 then
   # we have a definition, we're good to go
+  : # noop (true) statement for bash compatibility
 else
   echo >&2 "Missing CFLAGS_DYNAMIC inhibits building dynamic module lookup"
   exit 1