Subject: pkg/32458: mail/clamav -- tweaks to freshclamd rc.d script
To: None <pkg-manager@netbsd.org, gnats-admin@netbsd.org,>
From: None <svs+pr@grep.ru>
List: pkgsrc-bugs
Date: 01/05/2006 04:05:01
>Number:         32458
>Category:       pkg
>Synopsis:       mail/clamav -- tweaks to freshclamd rc.d script
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Thu Jan 05 04:05:00 +0000 2006
>Originator:     Sergey Svishchev
>Release:        3.99.10
>Organization:
>Environment:
>Description:
For some reason, the script creates the pid file itself, instead of using '-p' option to freshclamd.  sig_stop=KILL seems unnecessary, too.
>How-To-Repeat:

>Fix:
--- mail/clamav/files/freshclamd.sh	2 Nov 2004 00:55:20 -0000	1.2
+++ mail/clamav/files/freshclamd.sh	5 Jan 2006 03:59:39 -0000
@@ -5,21 +5,15 @@
 # PROVIDE: freshclamd
 # REQUIRE: DAEMON LOGIN clamd
 
-
 name="freshclamd"
 command="@PREFIX@/bin/freshclam"
 required_files="@PKG_SYSCONFDIR@/freshclam.conf"
 pidfile="@VARBASE@/run/${name}.pid"
-sig_stop="KILL"
 freshclamd_user="@CLAMAV_USER@"
 freshclamd_flags=${freshclamd_flags:=-c 2}
-command_args="-d"
+command_args="-d -p $pidfile"
 
 . /etc/rc.subr
 
 load_rc_config $name
 run_rc_command "$1"
-
-if [ "$1" != "stop" ]; then
-	echo $(check_process $command) > $pidfile
-fi