From owner-freebsd-rc@FreeBSD.ORG Sat Dec 24 12:30:15 2011 Return-Path: Delivered-To: freebsd-rc@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5CA74106566B for ; Sat, 24 Dec 2011 12:30:15 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 45AFA8FC0C for ; Sat, 24 Dec 2011 12:30:15 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id pBOCUF8C064101 for ; Sat, 24 Dec 2011 12:30:15 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id pBOCUF3h064098; Sat, 24 Dec 2011 12:30:15 GMT (envelope-from gnats) Date: Sat, 24 Dec 2011 12:30:15 GMT Message-Id: <201112241230.pBOCUF3h064098@freefall.freebsd.org> To: freebsd-rc@FreeBSD.org From: Maxim Ignatenko Cc: Subject: Re: conf/163508: [rc.subr] [patch] Add "enable" and "disable" commands to rc.subr X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Maxim Ignatenko 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: Sat, 24 Dec 2011 12:30:15 -0000 The following reply was made to PR conf/163508; it has been noted by GNATS. From: Maxim Ignatenko To: Doug Barton Cc: bug-followup@freebsd.org Subject: Re: conf/163508: [rc.subr] [patch] Add "enable" and "disable" commands to rc.subr Date: Sat, 24 Dec 2011 14:20:19 +0200 On 24 December 2011 04:15, Doug Barton wrote: > This idea has been considered before and rejected because it's too > difficult to catch all the corner cases, and actually editing a config > file is not really all that hard of a thing to do. > The idea was to make enabling/disabling services less error-prone. It don't need to catch _all_ corner cases, because if administrator do something unusual with startup configuration he should be able to manipulate it in proper way, or even have tools that do something similar. Proposed patch handles /etc/rc.conf, /etc/rc.conf.local and /etc/rc.conf.d/* properly (I hope), so it should fit nicely in 95% of cases. Doing `service someserive enable` is much faster and less error-prone that `service someservice rcvar ; echo someservicercvar_enable=YES >> /etc/rc.conf`