From nobody@FreeBSD.org  Mon Feb 21 09:37:05 2011
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 8793D106564A
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 21 Feb 2011 09:37:05 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from red.freebsd.org (red.freebsd.org [IPv6:2001:4f8:fff6::22])
	by mx1.freebsd.org (Postfix) with ESMTP id 75F908FC17
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 21 Feb 2011 09:37:05 +0000 (UTC)
Received: from red.freebsd.org (localhost [127.0.0.1])
	by red.freebsd.org (8.14.4/8.14.4) with ESMTP id p1L9b5Dm054770
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 21 Feb 2011 09:37:05 GMT
	(envelope-from nobody@red.freebsd.org)
Received: (from nobody@localhost)
	by red.freebsd.org (8.14.4/8.14.4/Submit) id p1L9b5WL054769;
	Mon, 21 Feb 2011 09:37:05 GMT
	(envelope-from nobody)
Message-Id: <201102210937.p1L9b5WL054769@red.freebsd.org>
Date: Mon, 21 Feb 2011 09:37:05 GMT
From: Taras <ds@ukrhub.net>
To: freebsd-gnats-submit@FreeBSD.org
Subject: handbook/install/chapter.sgml - repeated chunk of text
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         154931
>Category:       docs
>Synopsis:       handbook/install/chapter.sgml - repeated chunk of text
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    jkois
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Mon Feb 21 09:40:09 UTC 2011
>Closed-Date:    Wed Feb 23 10:44:15 UTC 2011
>Last-Modified:  Wed Feb 23 10:44:15 UTC 2011
>Originator:     Taras
>Release:        
>Organization:
>Environment:
>Description:
  doc/en_US.ISO8859-1/books/handbook/install/chapter.sgml - repeated chunk of text since rev. 1.417
  As of current revision (1.419) it can be found at the following lines:
1) 639-645, 647-648
vs.
2) 650-656, 661-663
>How-To-Repeat:
Check this output: http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/install-pre.html , find '2.3.7 Prepare the Boot Media', compare two <warning></warning> blocks.

>Fix:
skip lines 647-656 from *.sgml source

Patch attached with submission follows:

--- doc/en_US.ISO8859-1/books/handbook/install/chapter.sgml	2011-02-18 04:21:05.000000000 +0200
+++ doc/en_US.ISO8859-1/books/handbook/install/chapter.up.sgml	2011-02-21 11:24:01.000000000 +0200
@@ -644,17 +644,6 @@
 		may destroy your existing data.</para>
 	    </warning>
 
-	    <para>Set the <varname>kern.geom.debugflags</varname> sysctl to be
-	      able to write a master boot record to the target device.</para>
-
-	    <warning>
-	      <para>The example below
-		lists <filename class="devicefile">/dev/da0</filename> as the
-		target device where the image will be written. Be very careful
-		that you have the correct device as the output target, or you
-		may destroy your existing data.</para>
-	    </warning>
-
 	    <step>
 	      <title>Writing the Image with &man.dd.1;</title>
 


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-doc->jkois 
Responsible-Changed-By: jkois 
Responsible-Changed-When: Mon Feb 21 09:45:35 UTC 2011 
Responsible-Changed-Why:  
Take this one. 


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

From: Johann Kois <jkois@FreeBSD.org>
To: bug-followup@FreeBSD.org, ds.at.ukrhub.net@FreeBSD.org
Cc:  
Subject: Re: docs/154931: handbook/install/chapter.sgml - repeated chunk of
 text
Date: Mon, 21 Feb 2011 12:16:59 +0100

 Hm.  The whole section is a little bit "fishy".
 
 As mentioned the 2 warnings are "basically" the same, although the 
 wording is a little bit different.
 
 Another example: It says "Windows drive H: in the following example", 
 but H: is never ever mentioned again.  Actually there is no example at 
 all...
 
 I will take a closer look at that.
 
 jkois
 
 -- 
   Johann Kois
   jkois(at)FreeBSD.org
   FreeBSD Documentation Project
   FreeBSD German Documentation Project - https://doc.bsdgroup.de

From: Warren Block <wblock@wonkity.com>
To: To: ;
Cc:  
Subject: Subject: Re: docs/154931: handbook/install/chapter.sgml - repeated
 chunk of text
Date: Mon, 21 Feb 2011 07:44:18 -0700 (MST)

 I did reply to keramida@ earlier about this problem with 154837, but 
 neglected to include bug-followup@FreeBSD.org.  Here's the message, 
 along with a patch that fixes a couple of other problems.
 
 ---------- Forwarded message ----------
 Date: Thu, 17 Feb 2011 08:17:39
 From: Warren Block <wblock@wonkity.com>
 To: keramida@FreeBSD.org
 Subject: Re: docs/154837: [patch] update Handbook for writing memstick image
      with Windows
 
 On Thu, 17 Feb 2011, keramida@FreeBSD.org wrote:
 
 > Synopsis: [patch] update Handbook for writing memstick image with Windows
 > 
 > State-Changed-From-To: open->closed
 > State-Changed-By: keramida
 > State-Changed-When: Thu Feb 17 09:20:55 UTC 2011
 > State-Changed-Why:
 > Committed, in revision 1.417 of handbook/install/chapter.sgml
 > 
 > Thanks!
 > 
 > http://www.freebsd.org/cgi/query-pr.cgi?pr=154837
 
 The patch didn't quite apply as planned; there's a duplicate section at the 
 start of the FreeBSD procedure (two warnings, and the sysctl line). Possibly 
 due to the original having changed since the patch was created.
 
 Two other notes: the Windows section warns about the H: drive shown in the 
 example, which is not actually shown, and the link should probably go to the 
 project page, not a direct download that may be outdated.
 
 The following patch applies to the latest source to correct these three things.
 
 --- en_US.ISO8859-1/books/handbook/install/chapter.sgml.orig	2011-02-17 
 08:07:54.000000000 -0700
 +++ en_US.ISO8859-1/books/handbook/install/chapter.sgml	2011-02-17 
 08:12:24.000000000 -0700
 @@ -644,17 +644,6 @@
   		may destroy your existing data.</para>
   	    </warning>
 
 -	    <para>Set the <varname>kern.geom.debugflags</varname> sysctl to be
 -	      able to write a master boot record to the target device.</para>
 -
 -	    <warning>
 -	      <para>The example below
 -		lists <filename class="devicefile">/dev/da0</filename> as the
 -		target device where the image will be written. Be very careful
 -		that you have the correct device as the output target, or you
 -		may destroy your existing data.</para>
 -	    </warning>
 -
   	    <step>
   	      <title>Writing the Image with &man.dd.1;</title>
 
 @@ -679,11 +668,8 @@
   	    <title>Using &windows To Write the Image</title>
 
   	    <warning>
 -	      <para>The example below
 -	        lists <filename class="devicefile">H:</filename> as the drive
 -	        letter of the device where the image will be written.  Be very
 -	        careful that you have the correct device as the output target,
 -	        or you may destroy existing data.</para>
 +	      <para>Make sure you use the correct drive letter as the output
 +	        target, or you may overwrite and destroy existing data.</para>
   	    </warning>
 
   	    <step>
 @@ -691,8 +677,8 @@
 
   	      <para><application>Image Writer for Windows</application> is a
   	        free application that can correctly write an image file to a
 -	        memory stick.  Download it
 -	        from <ulink 
 url="http://launchpad.net/win32-image-writer/0.2/0.2/+download/win32diskimager-RELEASE-0.2-r23-win32.zip"></ulink>
 +	        memory stick.  Download it from
 +	        <ulink url="https://launchpad.net/win32-image-writer"></ulink>
   	        and extract it into a folder.</para>
   	    </step>
 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: docs/154931: commit references a PR
Date: Tue, 22 Feb 2011 20:57:46 +0000 (UTC)

 jkois       2011-02-22 20:57:42 UTC
 
   FreeBSD doc repository
 
   Modified files:
     en_US.ISO8859-1/books/handbook/install chapter.sgml 
   Log:
   PR:     docs/154931
   
   Fix some problems/hickups in the "Prepare Boot Media" section.
   
   Problem found by:       Taras (ds ett ukrhub dott net)
   Patch provided by:      Warren Block (wblock ett wonkity dott com)
   
   Revision  Changes    Path
   1.420     +4 -18     doc/en_US.ISO8859-1/books/handbook/install/chapter.sgml
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: open->closed 
State-Changed-By: jkois 
State-Changed-When: Wed Feb 23 10:43:15 UTC 2011 
State-Changed-Why:  
Changes to the handbook are online.  Close this PR. 

Thanks for pointing to this and also thanks for providing the patch to fix this. 


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