On 2011-01-26 at 21:13 +0100, Oliver Heesakkers wrote:
> You have to remove both the # and the space before LOOKUP_PASSWD=yes
> 
> So, "LOOKUP_PASSWD=yes" will work and " LOOKUP_PASSWD=yes" will fail.

*sigh*  Good catch, thanks.

I'll commit a change to permit leading spaces and tabs.  (Arbitrary
whitespace requires insufficiently portable regex syntax; it would be
nice to rely on pcregrep being available because we require PCRE, but
I'm not convinced that's true with all packaged installs.


diff --git a/src/scripts/lookups-Makefile b/src/scripts/lookups-Makefile
index 959d49e..38cda5a 100755
--- a/src/scripts/lookups-Makefile
+++ b/src/scripts/lookups-Makefile
@@ -34,12 +34,12 @@ tmp="$target.t"
 
 want_dynamic() {
   local dyn_name="$1"
-  grep -q "^LOOKUP_${dyn_name}[ $tab]*=[ $tab]*2" "$defs_source"
+  grep -q "^[ $tab]*LOOKUP_${dyn_name}[ $tab]*=[ $tab]*2" "$defs_source"
 }
 
 want_at_all() {
   local want_name="$1"
-  grep -q "^LOOKUP_${want_name}[ $tab]*=[ $tab]*." "$defs_source"
+  grep -q "^[ $tab]*LOOKUP_${want_name}[ $tab]*=[ $tab]*." "$defs_source"
 }
 
 emit_module_rule() {


-- 
## List details at http://lists.exim.org/mailman/listinfo/exim-users 
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/

Reply via email to