From owner-freebsd-rc@FreeBSD.ORG Mon Jun 1 06:08:34 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C6458106567C for ; Mon, 1 Jun 2009 06:08:34 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id 783D08FC1E for ; Mon, 1 Jun 2009 06:08:34 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: (qmail 30987 invoked by uid 399); 1 Jun 2009 06:08:29 -0000 Received: from localhost (HELO foreign.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 1 Jun 2009 06:08:29 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Message-ID: <4A23705C.3020603@FreeBSD.org> Date: Sun, 31 May 2009 23:08:28 -0700 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 2.0.0.21 (X11/20090423) MIME-Version: 1.0 To: Dmitry Morozovsky References: <200905301938.n4UJcpbF017191@svn.freebsd.org> <4A22CFA1.3050408@FreeBSD.org> In-Reply-To: X-Enigmail-Version: 0.95.7 OpenPGP: id=D5B2F0FB Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Pawel Jakub Dawidek , freebsd-rc@freebsd.org Subject: Re: svn commit: r193112 - head/etc/rc.d X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Jun 2009 06:08:35 -0000 Dmitry Morozovsky wrote: > On Sun, 31 May 2009, Doug Barton wrote: > > DB> Dmitry Morozovsky wrote: > DB> > Doug, > DB> > > DB> > On Sat, 30 May 2009, Doug Barton wrote: > DB> > > DB> > DB> Author: dougb > DB> > DB> Date: Sat May 30 19:38:51 2009 > DB> > DB> New Revision: 193112 > DB> > DB> URL: http://svn.freebsd.org/changeset/base/193112 > DB> > DB> > DB> > DB> Log: > DB> > DB> As previously advertised, remove this script prior to the 8.0 branch. > DB> > > DB> > Was there an agreement what should one do with dumping to gmirror (see > DB> > sbin/geom/class/mirror/gmirror.8) ? > DB> > DB> I'm not familiar with that issue, but it sounds like something that > DB> needs its own rc.d script. If someone who knows what is supposed to > DB> happen wants to write something up and send it to the freebsd-rc@ list > DB> I'll be glad to help review it. > > Something like (checks should be added, yeah) This is fine, but for consistency with the other stuff in rc.d the name of the script, the value of $name, and the PROVIDE should all be the same. I would name it something like gmsc and in etc/defaults/rc.conf (and probably in the script itself) add a comment indicating what it's for. Not sure what you mean by "checks should be added," I'm not familiar with gmirror at all. But it looks like you're on the right track, yes. Doug > #!/bin/sh > # > # $FreeBSD$ > # > > # BEFORE: savecore > # PROVIDE: gmirror-savecore > # KEYWORD: nojail > > . /etc/rc.subr > > name="gmirror_savecore" > start_cmd="gmsavecore_start" > stop_cmd=":" > > gmsavecore_start() > { > gmirror configure -b prefer /dev/dumpdev > } > > load_rc_config $name > run_rc_command "$1" > > possibly? > -- This .signature sanitized for your protection