From owner-freebsd-rc@FreeBSD.ORG Sat Jul 6 07:50:24 2013 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id DE28ABAA; Sat, 6 Jul 2013 07:50:24 +0000 (UTC) (envelope-from yaneurabeya@gmail.com) Received: from mail-pd0-x230.google.com (mail-pd0-x230.google.com [IPv6:2607:f8b0:400e:c02::230]) by mx1.freebsd.org (Postfix) with ESMTP id B247E1CAD; Sat, 6 Jul 2013 07:50:24 +0000 (UTC) Received: by mail-pd0-f176.google.com with SMTP id t12so2572862pdi.21 for ; Sat, 06 Jul 2013 00:50:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=references:mime-version:in-reply-to:content-type :content-transfer-encoding:message-id:cc:x-mailer:from:subject:date :to; bh=a79FSbpYNVDw3NBqlUouBRG+/PTuUQzZ2GVDgnvutv8=; b=YBhdURE9JsNfuaI06mWKLfIkeGon28OI6N1Is6cWud4p9cdK7o8aC0ew9j+iwxhw9a kpjRFgjrl6Et9Q4KErrNz7SIg1DxLKgNJRgVk94yG4FzGOIjyV1CexWJmQrJJyg5ShPj pfAHRJeqpLO7ABEXOIrn0mmI7VpW47F3T64HLSgVTmbu/5JhpABwUWYl90nW6cO2TCbN LBdgPlbeTCOKZC6PAxji/qxKNA141OauUGT3p45BsTdlJnK6VWO3USqr7pEmRvLfRSKN dEzIzGKgxidRDE4LZo7Ise0e5/5JpZ5Cq4nEfznyokf4e23vmtIDOuIAd28gP9GINhW/ qaYA== X-Received: by 10.66.226.111 with SMTP id rr15mr14443627pac.122.1373097024570; Sat, 06 Jul 2013 00:50:24 -0700 (PDT) Received: from [10.13.64.69] (mobile-166-147-080-076.mycingular.net. [166.147.80.76]) by mx.google.com with ESMTPSA id cx3sm11013775pbb.30.2013.07.06.00.50.22 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sat, 06 Jul 2013 00:50:23 -0700 (PDT) References: <201307060413.r664DmT5009602@svn.freebsd.org> <43915FB0-442B-42CA-BA1A-E346D95838B5@gmail.com> <13CA24D6AB415D428143D44749F57D7201FB2721@ltcfiswmsgmb21> Mime-Version: 1.0 (1.0) In-Reply-To: <13CA24D6AB415D428143D44749F57D7201FB2721@ltcfiswmsgmb21> Message-Id: X-Mailer: iPhone Mail (10B329) From: Garrett Cooper Subject: Re: svn commit: r252862 - head/usr.sbin Date: Sat, 6 Jul 2013 00:50:17 -0700 To: Devin Teske Content-Type: text/plain; charset=cp932 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.14 Cc: Devin Teske , "freebsd-rc@freebsd.org" X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.14 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: Sat, 06 Jul 2013 07:50:24 -0000 On Jul 5, 2013, at 11:05 PM, "Teske, Devin" wrot= e: > On Jul 5, 2013, at 10:09 PM, Garrett Cooper wrote: >=20 >> On Jul 5, 2013, at 9:13 PM, Devin Teske wrote: >>=20 >>> Author: dteske >>> Date: Sat Jul 6 04:13:47 2013 >>> New Revision: 252862 >>> URL: https://urldefense.proofpoint.com/v1/url?u=3Dhttp://svnweb.freebsd.= org/changeset/base/252862&k=3D%2FbkpAUdJWZuiTILCq%2FFnQg%3D%3D%0A&r=3DMrjs6v= R4%2Faj2Ns9%2FssHJjg%3D%3D%0A&m=3D6Emrz4%2BdiEiu3QIuKxkRkKl%2BdgggvTvDq79TFh= oaAC8%3D%0A&s=3Df8e3ea5c36067381ada1de66dd547b09eb051cd0761b399929dfa68851d0= ca37 >>> Log: >>> Take the training-wheels off, after nearly 30 months of development. MFC= to >>> stable/9 planned after MFC 3-day period. The MFC to stable/9 is desired f= or >>> the next release to get some much-needed time: >>> + Living side-by-side with sysinstall for compare/contrast/transition >>> + Living side-by-side with bsdinstall for integration/transition >>> + Additional feedback/testing before eventual 10.0-R to make it even bet= ter >>> MFC after: >>> 3 days >>=20 >> Uh, why did you remove the conditional..? Why not just change the default= from WITHOUT_BSDCONFIG to WITH_BSDCONFIG? >>=20 >> I don't need this necessarily on an already tuned system and this doesn't= seem like something that should always be included on an appliance=81c >=20 > One plans was to use the libraries I'm bringing in to solve this PR: >=20 > http://www.freebsd.org/cgi/query-pr.cgi?pr=3Dconf/163508 > "[rc.subr] [patch] Add "enable" and "disable" commands to rc.subr" >=20 > The initial patch was rejected by dougb and I (as can be seen in the audit= trail) because editing rc.conf(5) is not a simple proposition. bsdconfig(8)= brings in a shell library called "sysrc.subr" (and the sysrc(8) utility lev= erages it to provide all the nifty things it can do). The shell library is o= f interest if we want to implement the high-level concept from the PR: >=20 > sevice {name} { enable | disable | . . . } >=20 > Since sysrc.subr provides a simple "f_sysrc_set $var $value" syntax (I'll l= eave the rest up to your imagination). >=20 > Staying on-topic, bsdconfig (or rather, its libraries) could end up entwin= ed to the shell commands and you may end up using it without ever directly e= xecuting "bsdconfig". I'd like to read more about this. We (isilon) have hacked around rc(5) becau= se the performance of rc is serialized and poor. I would prefer to avoid add= ing more end-user bloat to rc because it will drive users and consumers to t= ake more drastic measures to bypass the rc system. Thanks..=