From mistral@imasy.or.jp  Mon Mar 20 00:43:31 2000
Return-Path: <mistral@imasy.or.jp>
Received: from tasogare.imasy.or.jp (tasogare.imasy.or.jp [202.227.24.5])
	by hub.freebsd.org (Postfix) with ESMTP id 0085D37B82B
	for <FreeBSD-gnats-submit@freebsd.org>; Mon, 20 Mar 2000 00:43:30 -0800 (PST)
	(envelope-from mistral@imasy.or.jp)
Received: from mistral.wind.prv (isdn0.imasy.or.jp [202.227.24.192])
	by tasogare.imasy.or.jp (8.9.3+3.2W/3.7W-tasogare/smtpfeed 1.01) with ESMTP id RAA06012
	for <FreeBSD-gnats-submit@freebsd.org>; Mon, 20 Mar 2000 17:43:17 +0900 (JST)
	(envelope-from mistral@imasy.or.jp)
Received: (from yohta@localhost)
	by mistral.wind.prv (8.9.3/3.7Wpl2-990626) id MAA02116;
	Mon, 20 Mar 2000 12:38:25 +0900 (JST)
Message-Id: <200003200843.RAA06012@tasogare.imasy.or.jp>
Date: Mon, 20 Mar 2000 12:38:25 +0900 (JST)
From: mistral@imasy.org
Sender: mistral@imasy.or.jp
Reply-To: mistral@imasy.org
To: FreeBSD-gnats-submit@freebsd.org
Subject: HTML -> ROFF convert problem in ntpdc.8
X-Send-Pr-Version: 3.2

>Number:         17500
>Category:       docs
>Synopsis:       HTML -> ROFF convert problem in ntpdc.8
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    sheldonh
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Mon Mar 20 00:50:01 PST 2000
>Closed-Date:    Mon Mar 20 03:52:39 PST 2000
>Last-Modified:  Mon Mar 20 03:54:19 PST 2000
>Originator:     Yoshihiko SARUMARU
>Release:        FreeBSD 4.0-RELEASE i386
>Organization:
Personal
>Environment:

	From 4.0-current (at the timing of NTP 4.0 was imported) to
	to HEAD

>Description:

	I here the man pages of NTP was converted from HTML documents.
	When I read down the man page of ntpdc(8), I found the same
	block with "pll", and the same and near position, sectioning is
	something wrong because of .It macro is misplaced by .Ic macro.
	There is one more problem that is not so critical. "reset" subtitle
	was not .Ic decorated and looks not so emphasized.

>How-To-Repeat:

	man ntpdc

>Fix:
	
	I propose a patch below
--- ntpdc.8.orig	Tue Feb 15 22:52:17 2000
+++ ntpdc.8	Mon Mar 20 12:26:18 2000
@@ -559,33 +559,12 @@
 .Qq mutlicastclient
 configuration option with default address.
 The default for this flag is disable.
-.It pll
-Enables
-the server to adjust its local clock by means of NTP.
-If disabled,
-the local clock free-runs at its intrinsic time and frequency
-offset.
-This flag is useful in case the local clock is controlled by
-some other device or protocol and
-NTP is used only to provide
-synchronization to other clients.
-In this case, the local clock driver is used.
-See the
-.Qo
-Reference Clock Drivers
-.Qc
-page
-(available as part of the HTML documentation
-provided in
-.Pa /usr/share/doc/ntp )
-for further information.
-The default for this flag is enable.
 .It monitor
 Enables the monitoring facility for the
 .Ic monlist
 command.
 The default for this flag is enable.
-.Ic pll
+.It pll
 Enables the server to adjust its local clock by means of NTP.
 If disabled,
 the local clock free-runs at its intrinsic time and frequency offset.
@@ -600,7 +579,7 @@
 .Qc
 page for further information.
 The default for this flag is enable.
-.Ic pps
+.It pps
 Enables the pulse-per-second (PPS) signal
 when frequency and time is disciplined
 by the precision time kernel modifications.
@@ -611,7 +590,7 @@
 page
 for further information.
 The default for this flag is disable.
-.Ic stats
+.It stats
 Enables the statistics facility.
 See the
 .Qq Monitoring Support
@@ -620,11 +599,11 @@
 page
 for further information.
 The default for this flag is enable.
-.Ic pll_kernel
+.It pll_kernel
 When the precision time kernel modifications are installed,
 this indicates the kernel controls the clock discipline;
 otherwise, the daemon controls the clock discipline.
-.Ic pps_kernel
+.It pps_kernel
 When the precision time kernel modifications are installed
 and a pulse-per-second (PPS) signal is available,
 this indicates the PPS signal controls the clock discipline;
@@ -697,7 +676,7 @@
 .Xc
 Clear a trap for asynchronous messages.
 See the source listing for further information.
-.It reset Ar counter Op Ar ...
+.It Ic reset Ar counter Op Ar ...
 Clear the statistics counters in various modules of the server.
 See the source listing for further information.
 .El



>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-doc->sheldonh 
Responsible-Changed-By: sheldonh 
Responsible-Changed-When: Mon Mar 20 03:42:20 PST 2000 
Responsible-Changed-Why:  
I did these manpages, so I'll take a look. 
State-Changed-From-To: open->closed 
State-Changed-By: sheldonh 
State-Changed-When: Mon Mar 20 03:52:39 PST 2000 
State-Changed-Why:  
You were absolutely right.  I used Ic instead of It by mistake.  I 
used your patch almost verbatim, but I kept the old style of 
referencing the HTML documentation for the first time. 

Committed to HEAD as rev 1.3,merged onto RELENG_4 as rev 1.2.2.1, 
thanks! 
>Unformatted:
