Subject: Re: install/34121
To: None <install-manager@netbsd.org, gnats-admin@netbsd.org,>
From: Peter Postma <peter@pointless.nl>
List: netbsd-bugs
Date: 10/12/2006 20:45:02
The following reply was made to PR install/34121; it has been noted by GNATS.

From: Peter Postma <peter@pointless.nl>
To: gnats-bugs@NetBSD.org
Cc: lukem@NetBSD.org
Subject: Re: install/34121
Date: Thu, 12 Oct 2006 22:40:08 +0200

 It seems that this message never reached gnats.
 Trying again.
 
 On Sat, Oct 07, 2006 at 05:08:44PM +0200, Peter Postma wrote:
 > I've attached a patch which checks for pf.conf and pf.os, but it can easily
 > overwrite your pf.conf if you're not careful.
 > 
 > I'm not sure if the patch is correct, Luke can you review it?
 > 
 > -- 
 > Peter Postma
 
 > Index: postinstall
 > ===================================================================
 > RCS file: /cvsroot/src/usr.sbin/postinstall/postinstall,v
 > retrieving revision 1.28
 > diff -u -r1.28 postinstall
 > --- postinstall	23 Sep 2006 08:27:52 -0000	1.28
 > +++ postinstall	7 Oct 2006 15:02:42 -0000
 > @@ -768,6 +768,33 @@
 >  }
 >  
 >  #
 > +#	pf
 > +#
 > +additem pf "pf configuration being up to date"
 > +do_pf()
 > +{
 > +	[ -n "$1" ] || err 2 "USAGE: do_pf  fix|check"
 > +	op=$1
 > +	failed=0
 > +
 > +	find_file_in_dirlist pf.conf "pf.conf" \
 > +	    ${SRC_DIR}/dist/pf/etc ${DEST_DIR}/etc \
 > +	    || return 1
 > +
 > +	find_file_in_dirlist pf.os "pf.os" \
 > +	    ${SRC_DIR}/dist/pf/etc ${DEST_DIR}/etc \
 > +	    || return 1
 > +
 > +	compare_dir $op ${dir} ${DEST_DIR}/etc 644 pf.conf
 > +	failed=$(( ${failed} + $? ))
 > +
 > +	compare_dir $op ${dir} ${DEST_DIR}/etc 644 pf.os
 > +	failed=$(( ${failed} + $? ))
 > +
 > +	return ${failed}
 > +}
 > +
 > +#
 >  #	postfix
 >  #
 >  additem postfix "/etc/postfix/ being up to date"
 
 
 -- 
 Peter Postma