From swear@blarg.net  Thu Mar 28 20:45:57 2002
Return-Path: <swear@blarg.net>
Received: from lists.blarg.net (lists.blarg.net [206.124.128.17])
	by hub.freebsd.org (Postfix) with ESMTP id 9803837B419
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 28 Mar 2002 20:45:56 -0800 (PST)
Received: from thig.blarg.net (thig.blarg.net [206.124.128.18])
	by lists.blarg.net (Postfix) with ESMTP id 2F1B9BD72
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 28 Mar 2002 20:45:56 -0800 (PST)
Received: from localhost.localdomain ([206.124.139.115])
	by thig.blarg.net (8.9.3/8.9.3) with ESMTP id UAA01203
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 28 Mar 2002 20:45:55 -0800
Received: (from jojo@localhost)
	by localhost.localdomain (8.11.6/8.11.3) id g2T4iTj99529;
	Thu, 28 Mar 2002 20:44:29 -0800 (PST)
	(envelope-from swear@blarg.net)
Message-Id: <m4r8m4f0eq.8m4@localhost.localdomain>
Date: 28 Mar 2002 20:44:29 -0800
From: "Gary W. Swearingen" <swear@blarg.net>
Reply-To: swear@blarg.net
To: FreeBSD-gnats-submit@freebsd.org
Subject: dd(1) manual has fatal spaces after "conv="
X-GNATS-Notify:

>Number:         36461
>Category:       docs
>Synopsis:       dd(1) manual has fatal spaces after "conv="
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    keramida
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Mar 28 20:50:01 PST 2002
>Closed-Date:    Tue Apr 09 19:12:18 PDT 2002
>Last-Modified:  Tue Apr 09 19:12:18 PDT 2002
>Originator:     Gary W. Swearingen
>Release:        FreeBSD 4.5-STABLE i386
>Organization:
none
>Environment:
n/a
================
>Description:

The dd(1) manual has extra spaces after the "conv=" operand syntax
line, which, if used in a command, is fatal.
================
>How-To-Repeat:

dd if=/dev/zero of=/dev/null conv= lcase
dd if=/dev/zero of=/dev/null conv=lcase, sparse
================
>Fix:

Change "conv= value[, value ...]" to "conv=value[,value,...]".

Maybe add to description "spaces are not allowed".
>Release-Note:
>Audit-Trail:

From: <trhodes@FreeBSD.org>
To: FreeBSD-gnats-submit@FreeBSD.org
Cc:  
Subject: Re: docs/36461: [PATCH] dd(1)
Date: Thu, 28 Mar 2002 23:27:41 -0800 (PST)

 Gary, this may be ``illegal'' mdoc, but could you tell me what you think
 about the following changes?  I have removed the old values and changed
 added new ones according to the pr report.  Little doubt is in my mind
 that you or ru, or Giorgos will say ``Hey Tom, there IS a backspace
 macro or something...  Oh well, the output looked ok when I did an
 nroff -man dd.1 on my machine...
 
 --
 Tom Rhodes
 
 
 --- dd.1.old	Fri Mar 29 02:11:03 2002
 +++ dd.1	Fri Mar 29 02:22:47 2002
 @@ -151,8 +151,7 @@
  For all other devices, the correct number of blocks is read without
  distinguishing between a partial or complete block being read.
  .It Xo
 -.Cm conv=
 -.Ns Cm value Ns Op \&, Cm value \&...
 +.Cm conv=value[,value,...]
  .Xc
  Where
  .Cm value

From: Giorgos Keramidas <keramida@freebsd.org>
To: "Gary W. Swearingen" <swear@blarg.net>
Cc: Tom Rhodes <trhodes@freebsd.org>, <bug-followup@freebsd.org>
Subject: Re: docs/36461: [PATCH] dd(1)
Date: Sat, 30 Mar 2002 04:00:16 +0200 (EET)

 On 2002-03-29 17:49, Gary W. Swearingen wrote:
 
 > (If I was going to bother ru at all, I'd just flat out ask him how to
 > implement that one line so he doesn't have to wade through the trivial
 > stuff.  But that's your business, of course.)
 
 If you (Gary or Tom, that is) prefer the arguments to in "italics",
 you will probably like the following:
 
 %%%
 Index: dd.1
 ===================================================================
 RCS file: /home/ncvs/src/bin/dd/dd.1,v
 retrieving revision 1.18
 diff -2 -u -r1.18 dd.1
 --- dd.1	15 Aug 2001 09:09:34 -0000	1.18
 +++ dd.1	29 Mar 2002 15:07:18 -0000
 @@ -152,6 +152,5 @@
  distinguishing between a partial or complete block being read.
  .It Xo
 -.Cm conv=
 -.Ns Cm value Ns Op \&, Cm value \&...
 +.Cm conv= Ns Cm value Ns Op \&, Ns Cm value Ns \&, Ns \&...
  .Xc
  Where
 %%%
 

From: swear@blarg.net (Gary W. Swearingen)
To: Giorgos Keramidas <keramida@freebsd.org>
Cc: Tom Rhodes <trhodes@freebsd.org>, <bug-followup@freebsd.org>
Subject: Re: docs/36461: [PATCH] dd(1)
Date: 29 Mar 2002 19:39:44 -0800

 Giorgos Keramidas <keramida@freebsd.org> writes:
 > 
 > If you (Gary or Tom, that is) prefer the arguments to in "italics",
 > you will probably like the following:
 ...
 > +.Cm conv= Ns Cm value Ns Op \&, Ns Cm value Ns \&, Ns \&...
 
 Well, the "before" version uses "Cm", but the other operands seem to use
 "Ar", so this seems better:
 
 .Cm conv= Ns Ar value Ns Op \&, Ns Ar value Ns \&, Ns \&...
 .Xc
 Where
 .Ar value

From: Tom Rhodes <darklogik@pittgoth.com>
To: swear@blarg.net (Gary W. Swearingen)
Cc: keramida@freebsd.org, trhodes@freebsd.org,
	bug-followup@freebsd.org
Subject: Re: docs/36461: [PATCH] dd(1)
Date: Sat, 30 Mar 2002 15:09:08 -0500

 On 29 Mar 2002 19:39:44 -0800
 swear@blarg.net (Gary W. Swearingen) wrote:
 
 > Giorgos Keramidas <keramida@freebsd.org> writes:
 > > 
 > > If you (Gary or Tom, that is) prefer the arguments to in
 > > "italics", you will probably like the following:
 
 Italics isn't really a huge deal in my opinion, the only REAL worry
 I had was markup.  Limited knowledge of mdoc(7) tells me to either
 have a more experianced person review my work before application into
 the system.  This is your call Gary, if you want it in italics just
 say so and Keramida can commit his patch.
 
 -- 
 Tom (Darklogik) Rhodes
 www.FreeBSD.org  -The Power To Serve
 www.Pittgoth.com -Pittgoth Discussion Portal
 trhodes@ {Pittgoth.com, FreeBSD.org}

From: swear@blarg.net (Gary W. Swearingen)
To: Tom Rhodes <darklogik@pittgoth.com>
Cc: keramida@freebsd.org, trhodes@freebsd.org,
	bug-followup@freebsd.org
Subject: Re: docs/36461: [PATCH] dd(1)
Date: 30 Mar 2002 19:09:44 -0800

 Tom Rhodes <darklogik@pittgoth.com> writes:
 
 > Italics isn't really a huge deal in my opinion, the only REAL worry
 > I had was markup.  Limited knowledge of mdoc(7) tells me to either
 > have a more experianced person review my work before application into
 > the system.  This is your call Gary, if you want it in italics just
 > say so and Keramida can commit his patch.
 
 I didn't know why italics was mentioned -- I never see it in manuals.
 But I just now read that ".Em" usually gives italics while I see
 underlined text from both ".Em" and ".Ar" (but bold text from ".Cm").
 
 I notice that the nslookup(8) manual uses Ar right of "=", but then it
 is describing interactive commands (.It Ic port= Ns Ar value).
 
 My call is Giorgos' last effort except with "Ar" on the right side of
 the "=" like the rest of the operands have, plus "Ar" on the
 corresponding reference to "value" in the description.  But I'll not
 complain if the committer wants it otherwise (as long as it has no
 spaces, of course).
State-Changed-From-To: open->closed 
State-Changed-By: keramida 
State-Changed-When: Sun Mar 31 12:50:10 PST 2002 
State-Changed-Why:  
This and other changes have been committed to -CURRENT. 
MFC coming soon... 


Responsible-Changed-From-To: freebsd-doc->keramida 
Responsible-Changed-By: keramida 
Responsible-Changed-When: Sun Mar 31 12:50:10 PST 2002 
Responsible-Changed-Why:  

http://www.freebsd.org/cgi/query-pr.cgi?pr=36461 
State-Changed-From-To: closed->patched 
State-Changed-By: keramida 
State-Changed-When: Sun Mar 31 12:51:57 PST 2002 
State-Changed-Why:  
Hmmm, my bad.  Until the MFC this should remain in 'patched' state. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=36461 

From: Tom Rhodes <darklogik@pittgoth.com>
To: swear@blarg.net (Gary W. Swearingen)
Cc: keramida@freebsd.org, trhodes@freebsd.org,
	bug-followup@freebsd.org
Subject: Re: docs/36461: [PATCH] dd(1)
Date: Sun, 31 Mar 2002 15:49:54 -0500

 On 30 Mar 2002 19:09:44 -0800
 swear@blarg.net (Gary W. Swearingen) wrote:
 
 > Tom Rhodes <darklogik@pittgoth.com> writes:
 > 
 > > Italics isn't really a huge deal in my opinion, the only REAL
 > > worry I had was markup.  Limited knowledge of mdoc(7) tells me to
 > > either have a more experianced person review my work before
 > > application into the system.  This is your call Gary, if you want
 > > it in italics just say so and Keramida can commit his patch.
 > 
 > I didn't know why italics was mentioned -- I never see it in
 > manuals. But I just now read that ".Em" usually gives italics while
 > I see underlined text from both ".Em" and ".Ar" (but bold text from
 > ".Cm").
 > 
 > I notice that the nslookup(8) manual uses Ar right of "=", but then
 > it is describing interactive commands (.It Ic port= Ns Ar value).
 > 
 > My call is Giorgos' last effort except with "Ar" on the right side
 > of the "=" like the rest of the operands have, plus "Ar" on the
 > corresponding reference to "value" in the description.  But I'll not
 > complain if the committer wants it otherwise (as long as it has no
 > spaces, of course).
 > 
 
 Yes, the final outcome must fix the original error ;)  I see that
 Giorgos has already taken the care to cc Ruslan on the matter, and
 we will go from there.  Although, I will continue to watch the
 developments of this issue to be sure that a resolution appears. 
 Thanks alot Gary.
 
 -- 
 Tom (Darklogik) Rhodes
 www.FreeBSD.org  -The Power To Serve
 www.Pittgoth.com -Pittgoth Discussion Portal
 trhodes@ {Pittgoth.com, FreeBSD.org}
State-Changed-From-To: patched->closed 
State-Changed-By: keramida 
State-Changed-When: Tue Apr 9 19:12:00 PDT 2002 
State-Changed-Why:  
Change has been MFC'ed. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=36461 
>Unformatted:
