From owner-freebsd-doc@FreeBSD.ORG Thu Oct 18 06:58:09 2012 Return-Path: Delivered-To: freebsd-doc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 1F778D18; Thu, 18 Oct 2012 06:58:09 +0000 (UTC) (envelope-from mmakonnen@gmail.com) Received: from mail-ee0-f54.google.com (mail-ee0-f54.google.com [74.125.83.54]) by mx1.freebsd.org (Postfix) with ESMTP id C19B78FC14; Thu, 18 Oct 2012 06:58:07 +0000 (UTC) Received: by mail-ee0-f54.google.com with SMTP id c50so4848598eek.13 for ; Wed, 17 Oct 2012 23:58:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=m2CJuNqq5KR5KydyKSrQMmaV8n6/I3P/ccMu+ypF4qU=; b=sv86brTHWX8hCN90Ttd4GFkVNL42xHH4A/67FICkRbXMKRf1Ug43WRsxI2+RG5fIJ4 T4BQ2sgI37Vqqk+Smbxj/miR1J1UUSbS/3YOYXGoGz0zZo4EpATTpS9Hlgi7rML33ruU sQnYfElgf/sCLj4fd4T+Ly4YAVfa2/akuTyEoXSJvZ8rFQ3qbsk7lX/p59SKdEJ4TnYC 2S7k9nbG+59NQzTIwKdsOanYkMylD35m1rEMvFgFCI+fzJ/K5aM5JQO6/tMJ3gpywrW8 PhXh/Na8qgZSyrnXIOh5n5OeiRnuWYPqTLpT65HQsvtP8k+Koc/Wsl8kQhOYKkYk0HvT KPFw== Received: by 10.14.194.136 with SMTP id m8mr29908734een.10.1350543486837; Wed, 17 Oct 2012 23:58:06 -0700 (PDT) Received: from [172.16.16.143] ([213.55.76.19]) by mx.google.com with ESMTPS id 7sm38300695eeg.5.2012.10.17.23.57.53 (version=SSLv3 cipher=OTHER); Wed, 17 Oct 2012 23:57:59 -0700 (PDT) Message-ID: <507FA86D.5040905@gmail.com> Date: Thu, 18 Oct 2012 09:57:49 +0300 From: Michael Telahun Makonnen User-Agent: Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/20121011 Thunderbird/16.0.1 MIME-Version: 1.0 To: Chris Rees Subject: Re: docs/172692: [PATCH] Bring parts of the rc scripting guides up to date References: <201210141319.q9EDJN6H085443@freefall.freebsd.org> <507C39D0.9030909@gmail.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: freebsd-doc@freebsd.org, freebsd-rc@freebsd.org, "bug-followup@freebsd.org" X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Oct 2012 06:58:09 -0000 On 10/15/2012 09:26 PM, Chris Rees wrote: >> Not sure why you felt this paragraph needed to be removed. > > Because the style is clearer and makes it much more obvious when a > variable is having its own default value set. In ports at least, the > current style is to use the less verbose form. > ok > >> While you are technically correct, I think you misunderstood the >> writer's intent, which was to show how an rc.conf(8) variable can be >> used in a subroutine to control the behavior of the command. I agree >> that the example isn't a very good one (in that it doesn't depict a >> valid use case), but I think the "spirit" is correct. Maybe you can >> suggest a better example? >> > > I think that this script is very simple by design, and making a better > example would complicate it. It is definitely worth pointing out the > alternative though; it makes useful food for thought; both examples > with a disclaimer. ok > > [1] http://www.bayofrum.net/~crees/patches/rc-scripting-modernise2.diff > Looks fine to me. Thanks for taking the time to update the documentation. Cheers, Mike.