On Mon, Feb 11, 2002 at 03:39:15PM +0200, Martti Kuparinen wrote:
  | 
  | Module Name:	basesrc
  | Committed By:	martti
  | Date:		Mon Feb 11 13:39:14 UTC 2002
  | 
  | Modified Files:
  | 	basesrc/etc/rc.d: ipfs
  | 
  | Log Message:
  | Don't try to execute this script if ipfs is not set to YES. Otherwise
  | you'll see the following message:
  | 
  |    # /etc/rc.d/ipfs stop
  |    /etc/rc.d/ipfs: WARNING: $ipfs is not set properly.
  | 
  | This horrible change is needed because of the "shutdown" keyword.

Your change does not seem correct to me, and I've fixed it.

rc.subr will only generate that warning if you don't have ipfs set to
yes or no.  Is your /etc/defaults/rc.conf up to date (and thus
contains "ipfs=NO") ?