From owner-freebsd-perl@FreeBSD.ORG  Fri Nov 26 14:37:09 2004
Return-Path: <owner-freebsd-perl@FreeBSD.ORG>
Delivered-To: freebsd-perl@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 48E0916A4CE
	for <perl@FreeBSD.org>; Fri, 26 Nov 2004 14:37:09 +0000 (GMT)
Received: from hub.xunix.org (hub.xunix.org [163.5.255.82])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 0EFD143D54
	for <perl@FreeBSD.org>; Fri, 26 Nov 2004 14:37:09 +0000 (GMT)
	(envelope-from flz@xbsd.org)
Received: from localhost (localhost [127.0.0.1])
	by hub.xunix.org (Postfix) with ESMTP id 9031CD7BAA
	for <perl@FreeBSD.org>; Fri, 26 Nov 2004 15:34:25 +0100 (CET)
Received: from hub.xunix.org ([127.0.0.1])
 by localhost (hub [127.0.0.1]) (amavisd-new, port 10024) with ESMTP
 id 14742-08 for <perl@FreeBSD.org>; Fri, 26 Nov 2004 15:34:24 +0100 (CET)
Received: from xbsd.org (localhost [127.0.0.1])
	by hub.xunix.org (Postfix) with SMTP id 399D4D79A3
	for <perl@FreeBSD.org>; Fri, 26 Nov 2004 15:34:24 +0100 (CET)
Received: from 163.5.255.82 (proxying for 10.226.6.5)
        (SquirrelMail authenticated user flz@xbsd.org)
        by webmail.xunix.org with HTTP;
        Fri, 26 Nov 2004 15:34:24 +0100 (CET)
Message-ID: <43830.163.5.255.82.1101479664.squirrel@webmail.xunix.org>
Date: Fri, 26 Nov 2004 15:34:24 +0100 (CET)
From: "Florent Thoumie" <flz@xbsd.org>
To: <perl@FreeBSD.org>
X-Priority: 3
Importance: Normal
X-MSMail-Priority: Normal
X-Mailer: SquirrelMail (version 1.2.6)
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 8bit
X-Virus-Scanned: by amavisd-new-20030616-p3 (Debian) at xunix.org
Subject: mail/p5-Mail-SpamAssassin: %%SQL%% not replaced in sa-spamd.sh when
	WITH_SQLDB not defined
X-BeenThere: freebsd-perl@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: maintainer of a number of perl-related ports
	<freebsd-perl.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-perl>,
	<mailto:freebsd-perl-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-perl>
List-Post: <mailto:freebsd-perl@freebsd.org>
List-Help: <mailto:freebsd-perl-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-perl>,
	<mailto:freebsd-perl-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 26 Nov 2004 14:37:09 -0000

I guess :

.if defined(WITH_SQLDB)
SED_SCRIPT+=    -e 's|%%SQL%%|-Q|g'
.endif

should replaced by

.if defined(WITH_SQLDB)
SED_SCRIPT+=    -e 's|%%SQL%%|-Q|g'
.else
SED_SCRIPT+=    -e 's|%%SQL%%||-g'
.endif

I'm not sure this is really harmful but eyes focused on %%SQL%% in ps aux
output, so this is pure style :)
-- 
Florent Thoumie
flz@xbsd.org