From: Adam D. Barratt Date: Tue, 24 Sep 2019 19:48:22 +0000 (+0100) Subject: eximconf: use a more idiomatic way of "resetting" a variable X-Git-Url: https://git.adam-barratt.org.uk/?p=mirror%2Fdsa-puppet.git;a=commitdiff_plain;h=86bb13d735169395de039abf269bf9bf8d0fdd72 eximconf: use a more idiomatic way of "resetting" a variable An empty string is as valid as any other, so there's no need for hacks. Signed-off-by: Adam D. Barratt --- diff --git a/modules/exim/templates/eximconf.erb b/modules/exim/templates/eximconf.erb index 858c94bc2..b4363a640 100644 --- a/modules/exim/templates/eximconf.erb +++ b/modules/exim/templates/eximconf.erb @@ -293,10 +293,11 @@ acl_spamlovers: deny acl_getprofile: - # This is a bad hack to reset the variable, by defining it be something - # never referenced. - warn set acl_m_rprf = $acl_m_undefined + # Determine the mail profile for this recipient. + # An empty string implies no match has been found. + + warn set acl_m_rprf = warn recipients = survey@popcon.debian.org set acl_m_rprf = PopconMail