ports/90708: Fixing for apache starting shell script ?

[Joe Horn]

>Number:         90708
>Category:       ports
>Synopsis:       Fixing for apache starting shell script ?
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Dec 20 20:00:17 GMT 2005
>Closed-Date:
>Last-Modified:
>Originator:     Joe Horn
>Release:        5.4-RELEASE-p8
>Organization:
MI, CHU, Taiwan
>Environment:
FreeBSD Power.mi.chu.edu.tw 5.4-RELEASE-p8 FreeBSD 5.4-RELEASE-p8 #9: Thu Oct 13 03:30:02 CST 2005     root at Power.mi.chu.edu.tw:/usr/obj/usr/src/sys/Power  i386
>Description:
I installed Apache 2.2 ( /usr/ports/www/apache22 ) on this machine.

In some day past , I added this line in /boot/loader.conf :
  accf_http_load="YES"

But I found that newer version ( apache-2.2.0_4 ) support this option in /etc/rc.conf :
  apache22_http_accept_enable="YES"
So I added, and remove the line below in /boot/loader.conf .

After I use this command for starting apache :
  /usr/local/etc/rc.d/apache22.sh start

The screen shows these messages :
  kldstat: illegal option -- q
  usage: kldstat [-v] [-i id] [-n name]
  kldload: can't load accf_http: File exists
  Performing sanity check on apache22 configuration:
  Syntax OK
  kldstat: illegal option -- q
  usage: kldstat [-v] [-i id] [-n name]
  kldload: can't load accf_http: File exists
  Stopping apache22.
  Waiting for PIDS: 57769.
  kldstat: illegal option -- q
  usage: kldstat [-v] [-i id] [-n name]
  kldload: can't load accf_http: File exists
  Starting apache22.

Because the options '-q' and '-m' are supported in FreeBSD6 , but not in FreeBSD5 .
I modified this line in /usr/local/etc/rc.d/apache22.sh :
        if ! /sbin/kldstat -q -m accf_http
with :
        if ! /sbin/kldstat -n accf_http

Ths messages shown after the command '/usr/local/etc/rc.d/apache22.sh start' seems to be OK now :
  Id Refs Address    Size     Name
   2    1 0xc09fe000 2280     accf_http.ko
  Performing sanity check on apache22 configuration:
  Syntax OK
  Id Refs Address    Size     Name
   2    1 0xc09fe000 2280     accf_http.ko
  Stopping apache22.
  Waiting for PIDS: 4588.
  Id Refs Address    Size     Name
   2    1 0xc09fe000 2280     accf_http.ko
  Starting apache22.

My solution is correct, isn't it?
Thanks, and sorry for my poor english...
>How-To-Repeat:
              
>Fix:
              
>Release-Note:
>Audit-Trail:
>Unformatted: