Re: [exim] exim-4.74 in Suse 11.3

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() {