From owner-cvs-src@FreeBSD.ORG Wed Sep 27 10:50:24 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2709516A412; Wed, 27 Sep 2006 10:50:24 +0000 (UTC) (envelope-from joel@FreeBSD.org) Received: from av9-1-sn2.hy.skanova.net (av9-1-sn2.hy.skanova.net [81.228.8.179]) by mx1.FreeBSD.org (Postfix) with ESMTP id 575AB43D46; Wed, 27 Sep 2006 10:50:23 +0000 (GMT) (envelope-from joel@FreeBSD.org) Received: by av9-1-sn2.hy.skanova.net (Postfix, from userid 502) id 1E6E0381BE; Wed, 27 Sep 2006 12:50:22 +0200 (CEST) Received: from smtp4-1-sn2.hy.skanova.net (smtp4-1-sn2.hy.skanova.net [81.228.8.92]) by av9-1-sn2.hy.skanova.net (Postfix) with ESMTP id 0C7AF37E75; Wed, 27 Sep 2006 12:50:22 +0200 (CEST) Received: from dude.automatvapen.se (81-229-112-193-no21.tbcn.telia.com [81.229.112.193]) by smtp4-1-sn2.hy.skanova.net (Postfix) with ESMTP id C7D0537E45; Wed, 27 Sep 2006 12:50:21 +0200 (CEST) From: Joel Dahl To: Christian Brueffer In-Reply-To: <20060926202339.GA2039@haakonia.hitnet.RWTH-Aachen.DE> References: <200609261959.k8QJxqkh068350@repoman.freebsd.org> <20060926202339.GA2039@haakonia.hitnet.RWTH-Aachen.DE> Content-Type: text/plain Date: Wed, 27 Sep 2006 12:50:21 +0200 Message-Id: <1159354221.670.10.camel@localhost> Mime-Version: 1.0 X-Mailer: Evolution 2.6.1 FreeBSD GNOME Team Port Content-Transfer-Encoding: 7bit Cc: cvs-src@FreeBSD.org, Daniel Gerzo , src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/share/examples/mdoc example.4 X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 27 Sep 2006 10:50:24 -0000 On Tue, 2006-09-26 at 22:23 +0200, Christian Brueffer wrote: > On Tue, Sep 26, 2006 at 07:59:52PM +0000, Daniel Gerzo wrote: > > danger 2006-09-26 19:59:52 UTC > > > > FreeBSD src repository (doc committer) > > > > Modified files: > > share/examples/mdoc example.4 > > Log: > > Remove second person from the sentece and rephrase a bit. > > > > Approved by: trhodes (mentor), keramida (mentor) > > > > Revision Changes Path > > 1.26 +6 -8 src/share/examples/mdoc/example.4 > > I don't agree to these changes, see below for details (also I don't see where > exactly you removed second person usage, it's still being used in the new > version). > > > | =================================================================== > > | RCS file: /usr/local/www/cvsroot/FreeBSD/src/share/examples/mdoc/example.4,v > > | retrieving revision 1.25 > > | retrieving revision 1.26 > > | diff -u -p -r1.25 -r1.26 > > | --- src/share/examples/mdoc/example.4 2006/09/18 15:18:54 1.25 > > | +++ src/share/examples/mdoc/example.4 2006/09/26 19:59:52 1.26 > > | @@ -22,7 +22,7 @@ > > | .\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF > > | .\" SUCH DAMAGE. > > | .\" > > | -.\" $FreeBSD: /usr/local/www/cvsroot/FreeBSD/src/share/examples/mdoc/example.4,v 1.25 2006/09/18 15:18:54 ru Exp $ > > | +.\" $FreeBSD: /usr/local/www/cvsroot/FreeBSD/src/share/examples/mdoc/example.4,v 1.26 2006/09/26 19:59:52 danger Exp $ > > | .\" > > | .\" Note: The date here should be updated whenever a non-trivial > > | .\" change is made to the manual page. > > | @@ -33,11 +33,9 @@ > > | .Nm example > > | .Nd "example device driver manual page" > > | .Sh SYNOPSIS > > | -To compile the > > | -.Ns Nm > > | -driver into the kernel, > > | -place the following lines in the > > | -kernel configuration file: > > | +To enable support for > > | +.Ns Nm , > > | +place the following lines in the kernel configuration file: > > The formulation used before was much more accurate WRT the distinction > we make between compiling something into the kernel and loading it as a > module. If we load something as a module we also "enable support for > it". Agreed. The previous statement seems more clear and accurate. > > | .Bd -ragged -offset indent > > | .Cd "device example" > > | .Cd "options EXAMPLE_DEBUG" > > | @@ -45,9 +43,9 @@ kernel configuration file: > > | .Pp > > | Alternatively, to load the > > | .Ns Nm > > | -driver as a > > | -module at boot time, place the following line in > > | -.Xr loader.conf 5 : > > | +as a module at boot time, add the following line into the > > | +.Xr loader.conf 5 > > | +file: > > | .Bd -literal -offset indent > > | example_load="YES" > > | .Ed > > > > Removing "driver" here is wrong. "...to load the .Nm..." what, the .Nm > driver? The .Nm utility? It's just incorrect to rely on context here > and it makes the sentence sound really awkward. > > IMHO the SYNOPSIS section in section 4 manpages is kind of a standard > now, as it's being used in the majority of section 4 manpages. > Changes to it should not be made without _good_ reason and with a sweep > through all manpages that use it. Among other things, the purpose of > this section is to achieve consistency across section 4 manpages. Agreed, this change is just plain wrong. I've read the discussion and I believe this commit should be backed out. -- Joel