NetBSD Problem Report #39958

From www@NetBSD.org  Wed Nov 19 13:16:45 2008
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [204.152.190.11])
	by narn.NetBSD.org (Postfix) with ESMTP id 71DA763B8BD
	for <gnats-bugs@gnats.netbsd.org>; Wed, 19 Nov 2008 13:16:45 +0000 (UTC)
Message-Id: <20081119131645.13A5763B874@narn.NetBSD.org>
Date: Wed, 19 Nov 2008 13:16:45 +0000 (UTC)
From: <nb80@internode.on.net>
Reply-To: <nb80@internode.on.net>
To: gnats-bugs@NetBSD.org
Subject: amdtemp(4) man page: wording and wrong reference
X-Send-Pr-Version: www-1.0

>Number:         39958
>Category:       misc
>Synopsis:       amdtemp(4) man page: wording and wrong reference
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    jnemeth
>State:          closed
>Class:          doc-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Nov 19 13:20:00 +0000 2008
>Closed-Date:    Sat Nov 22 09:29:23 +0000 2008
>Last-Modified:  Sat Nov 22 09:29:23 +0000 2008
>Originator:     Joerg Niendorf
>Release:        5.0_BETA
>Organization:
none
>Environment:
N/A
>Description:
In the DESCRIPTION section of the amdtemp(4) man page, there is a "be" that does not belong there: "...and there are be up to two cores per CPU
socket."

Under "SEE ALSO", there is a reference to an envstat(4) man page that
does not exist. I think that should be envstat(8) instead?!

Also, the envsys(4) man page could probably be listed in the "SEE ALSO"
section, too.

And finally, I think the wording of the second sentence in "HISTORY"
could be improved. Please refer to the below diff.

>How-To-Repeat:
man amdtemp
>Fix:
Index: amdtemp.4
===================================================================
RCS file: /cvsroot/src/share/man/man4/amdtemp.4,v
retrieving revision 1.1
diff -b -u -r1.1 amdtemp.4
--- amdtemp.4	22 Apr 2008 22:37:13 -0000	1.1
+++ amdtemp.4	19 Nov 2008 11:59:33 -0000
@@ -43,7 +43,7 @@
 These sensors were officially introduced in AMD K8 Revision F processors,
 and provide 0.5 degC accuracy.  Precision was improved in Revision
 G chips, which provide two more bits for 0.25 degC steppings.  Each core
-has two temperature sensors, and there are be up to two cores per CPU socket.
+has two temperature sensors, and there are up to two cores per CPU socket.
 .Pp
 AMD Barcelona, AMD Phenom and AMD Griffin provide 0.125 degC accuracy
 and provide one temperature sensor for each CPU socket.
@@ -58,7 +58,8 @@
 .It Li "CPUN sensor0" Ta "uK" Ta "cpuN temperature"
 .El
 .Sh SEE ALSO
-.Xr envstat 4 ,
+.Xr envsys 4 ,
+.Xr envstat 8 ,
 .Xr powerd 8
 .Sh HISTORY
 The
@@ -67,7 +68,7 @@
 .Ox 4.4
 named
 .Xr kate 4 .
-And then was ported to
+It was then ported to
 .Nx 5.0 .
 The driver has been renamed with support for newer AMD CPUs.
 .Sh AUTHORS

>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: misc-bug-people->jnemeth
Responsible-Changed-By: jnemeth@NetBSD.org
Responsible-Changed-When: Thu, 20 Nov 2008 00:38:51 +0000
Responsible-Changed-Why:
I'm working on it.  Just have a question on how to handle the Xref to
kate(4) since there is no such device in NetBSD.


From: John Nemeth <jnemeth@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/39958 CVS commit: src/share/man/man4
Date: Thu, 20 Nov 2008 10:09:06 +0000 (UTC)

 Module Name:	src
 Committed By:	jnemeth
 Date:		Thu Nov 20 10:09:06 UTC 2008

 Modified Files:
 	src/share/man/man4: amdtemp.4

 Log Message:
 PR/39958 - Joerg Niendorf -- grammar and Xref fixes


 To generate a diff of this commit:
 cvs rdiff -r1.2 -r1.3 src/share/man/man4/amdtemp.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->pending-pullups
State-Changed-By: jnemeth@NetBSD.org
State-Changed-When: Thu, 20 Nov 2008 10:34:24 +0000
State-Changed-Why:
Fixes have been commiteed.  This is now waiting on [pullup-5 #98] amdtemp.4 .


From: Soren Jacobsen <snj@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/39958 CVS commit: [netbsd-5] src/share/man/man4
Date: Sat, 22 Nov 2008 04:38:08 +0000 (UTC)

 Module Name:	src
 Committed By:	snj
 Date:		Sat Nov 22 04:38:08 UTC 2008

 Modified Files:
 	src/share/man/man4 [netbsd-5]: amdtemp.4

 Log Message:
 Pull up following revision(s) (requested by jnemeth in ticket #98):
 	share/man/man4/amdtemp.4: revision 1.3
 PR/39958 - Joerg Niendorf -- grammar and Xref fixes


 To generate a diff of this commit:
 cvs rdiff -r1.1.8.1 -r1.1.8.2 src/share/man/man4/amdtemp.4

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: pending-pullups->closed
State-Changed-By: jnemeth@NetBSD.org
State-Changed-When: Sat, 22 Nov 2008 09:29:23 +0000
State-Changed-Why:
Pullups completed.  Thanks for the PR!


>Unformatted:

Home	PR Database Search