Bug 142418

Summary: [PATCH] newfs_msdos(8) doc fix
Product: Documentation Reporter: David Naylor <naylor.b.david>
Component: Books & ArticlesAssignee: Benedict Reuschling <bcr>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description David Naylor 2010-01-07 08:50:04 UTC 
Documentation says bytes per sector are valid from 128 but the program says 512

Fix: See patch

Patch attached with submission follows:
How-To-Repeat: (Well, -S doesn't work to well, in my experience but:)
newfs_msdos -S 256 /dev/null
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2010-01-07 15:30:52 UTC 
Responsible Changed
From-To: freebsd-bugs->freebsd-doc

reclassify.
Comment 2 Benedict Reuschling freebsd_committer freebsd_triage 2010-07-25 12:16:57 UTC 
Responsible Changed
From-To: freebsd-doc->bcr

I'll handle this.
Comment 3 dfilter service freebsd_committer freebsd_triage 2010-07-25 12:31:44 UTC 
Author: bcr (doc committer)
Date: Sun Jul 25 11:31:29 2010
New Revision: 210463
URL: http://svn.freebsd.org/changeset/base/210463

Log:
  Clarify that the "number of bytes per sector"-range in the
  -S option is meant to be "inclusive".
  
  The original issue of the PR was already fixed.
  
  PR:                 docs/142418
  Submitted by:       David Naylor (naylor dot b dot david at gmail dot com)
  No objection from:  kib
  MFC after:          5 days

Modified:
  head/sbin/newfs_msdos/newfs_msdos.8

Modified: head/sbin/newfs_msdos/newfs_msdos.8
==============================================================================
--- head/sbin/newfs_msdos/newfs_msdos.8	Sun Jul 25 10:06:56 2010	(r210462)
+++ head/sbin/newfs_msdos/newfs_msdos.8	Sun Jul 25 11:31:29 2010	(r210463)
@@ -25,7 +25,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd April 11, 2009
+.Dd July 25, 2010
 .Dt NEWFS_MSDOS 8
 .Os
 .Sh NAME
@@ -116,7 +116,7 @@ The default is
 .It Fl S Ar sector-size
 Number of bytes per sector.
 Acceptable values are powers of 2
-in the range 512 through 32768.
+in the range 512 through 32768, inclusive.
 .It Fl a Ar FAT-size
 Number of sectors per FAT.
 .It Fl b Ar block-size
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
Comment 4 Benedict Reuschling freebsd_committer freebsd_triage 2010-07-25 13:14:18 UTC 
State Changed
From-To: open->patched

A corrected version was already committed, we just 
added the "inclusive" part. Leave this in patched state 
until the MFC is finished.
Comment 5 dfilter service freebsd_committer freebsd_triage 2010-07-30 17:24:00 UTC 
Author: bcr (doc committer)
Date: Fri Jul 30 16:23:45 2010
New Revision: 210659
URL: http://svn.freebsd.org/changeset/base/210659

Log:
  MFC r210463:
  
  Clarify that the "number of bytes per sector"-range in the
  -S option is meant to be "inclusive".
  
  The original issue of the PR was already fixed.
  
  PR:                 docs/142418
  Submitted by:       David Naylor (naylor dot b dot david at gmail dot com)
  No objection from:  kib

Modified:
  stable/8/sbin/newfs_msdos/newfs_msdos.8
Directory Properties:
  stable/8/sbin/newfs_msdos/   (props changed)

Modified: stable/8/sbin/newfs_msdos/newfs_msdos.8
==============================================================================
--- stable/8/sbin/newfs_msdos/newfs_msdos.8	Fri Jul 30 15:56:00 2010	(r210658)
+++ stable/8/sbin/newfs_msdos/newfs_msdos.8	Fri Jul 30 16:23:45 2010	(r210659)
@@ -25,7 +25,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd April 11, 2009
+.Dd July 25, 2010
 .Dt NEWFS_MSDOS 8
 .Os
 .Sh NAME
@@ -116,7 +116,7 @@ The default is
 .It Fl S Ar sector-size
 Number of bytes per sector.
 Acceptable values are powers of 2
-in the range 512 through 32768.
+in the range 512 through 32768, inclusive.
 .It Fl a Ar FAT-size
 Number of sectors per FAT.
 .It Fl b Ar block-size
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
Comment 6 Benedict Reuschling freebsd_committer freebsd_triage 2010-07-30 17:59:02 UTC 
State Changed
From-To: patched->closed

MFC is done, PR closed. Thanks for sending the patch.