[exim-dev] possible bug in lookups-Makefile config script

Αρχική Σελίδα
Delete this message
Reply to this message
Συντάκτης: Tony Finch
Ημερομηνία:  
Προς: exim-dev
Αντικείμενο: [exim-dev] possible bug in lookups-Makefile config script
I noticed that `make configure` complained:

Missing CFLAGS_DYNAMIC inhibits building dynamic module lookup

although Makefile-Linux sets it. I think the patch below is a plausible
fix; have I overlooked any subtlety?

Tony.
--
<fanf@???> <dot@???> http://dotat.at/ ${sg{\N${sg{\
N\}{([^N]*)(.)(.)(.*)}{\$1\$3\$2\$1\$3\n\$2\$3\$4\$3\n\$3\$2\$4}}\
\N}{([^N]*)(.)(.)(.*)}{\$1\$3\$2\$1\$3\n\$2\$3\$4\$3\n\$3\$2\$4}}


>From 87233b77b5b46f75f301f42804858209e99d2560 Mon Sep 17 00:00:00 2001

From: Tony Finch <dot@???>
Date: Thu, 8 Nov 2012 19:38:00 +0000
Subject: [PATCH] lookups-Makefile: accept ?= operator when checking for CFLAGS_DYNAMIC

Makefile-Linux sets CFLAGS_DYNAMIC using ?= so the lookups-Makefile
configuration script needs to understand it.
---
 src/scripts/lookups-Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


diff --git a/src/scripts/lookups-Makefile b/src/scripts/lookups-Makefile
index 14c1525..6e6342c 100755
--- a/src/scripts/lookups-Makefile
+++ b/src/scripts/lookups-Makefile
@@ -44,7 +44,7 @@ export LC_ALL
# nb: do not permit leading whitespace for this, as CFLAGS_DYNAMIC is exported
# to the lookups subdir via a line with leading whitespace which otherwise
# matches
-if grep -q "^CFLAGS_DYNAMIC[ $tab]*=" "$defs_source"
+if grep -q "^CFLAGS_DYNAMIC[ $tab?]*=" "$defs_source"
then
# we have a definition, we're good to go
enable_dynamic=yes
--
1.7.3.GIT