From owner-freebsd-doc@FreeBSD.ORG  Thu Apr 17 16:10:16 2003
Return-Path: <owner-freebsd-doc@FreeBSD.ORG>
Delivered-To: freebsd-doc@hub.freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 33DB737B401
	for <freebsd-doc@hub.freebsd.org>;
	Thu, 17 Apr 2003 16:10:16 -0700 (PDT)
Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21])
	by mx1.FreeBSD.org (Postfix) with ESMTP id B19A343FCB
	for <freebsd-doc@hub.freebsd.org>;
	Thu, 17 Apr 2003 16:10:15 -0700 (PDT)
	(envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1])
	by freefall.freebsd.org (8.12.9/8.12.9) with ESMTP id h3HNAFUp047638
	for <freebsd-doc@freefall.freebsd.org>;
	Thu, 17 Apr 2003 16:10:15 -0700 (PDT)
	(envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
	by freefall.freebsd.org (8.12.9/8.12.9/Submit) id h3HNAF0x047637;
	Thu, 17 Apr 2003 16:10:15 -0700 (PDT)
Date: Thu, 17 Apr 2003 16:10:15 -0700 (PDT)
Message-Id: <200304172310.h3HNAF0x047637@freefall.freebsd.org>
To: freebsd-doc@FreeBSD.org
From: Martin Heinen <martin@sumuk.de>
Subject: Re: docs/51109: [PATCH] handbook (kernelconfig)
X-BeenThere: freebsd-doc@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
Reply-To: Martin Heinen <martin@sumuk.de>
List-Id: Documentation project <freebsd-doc.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-doc>,
	<mailto:freebsd-doc-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-doc>
List-Post: <mailto:freebsd-doc@freebsd.org>
List-Help: <mailto:freebsd-doc-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-doc>,
	<mailto:freebsd-doc-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 17 Apr 2003 23:10:16 -0000

The following reply was made to PR docs/51109; it has been noted by GNATS.

From: Martin Heinen <martin@sumuk.de>
To: Martin Karlsson <mk-freebsd@bredband.net>
Cc: FreeBSD-gnats-submit@freebsd.org
Subject: Re: docs/51109: [PATCH] handbook (kernelconfig)
Date: Fri, 18 Apr 2003 01:08:57 +0200

 On Fri, Apr 18, 2003 at 12:16:31AM +0200, Martin Karlsson wrote:
 
 > >Description:
 > The handbook does explain how to copy GENERIC to MYKERNEL, and edit
 > MYKERNEL, but does not explicitly mention why editing
 > GENERIC directly is a bad idea.  Perhaps it should? This patch aims to
 > fix that.
 > 
 > I have seen related questions (Where did GENERIC go?) on -questions only
 > a few times, but being explicit about important things is rarely bad IMO.
 > 
 > >How-To-Repeat:
 > Read the handbook, 9.3 Building and Installing a Custom Kernel.
 > >Fix:
 > 
 > 	
 > 
 > --- kernelconfig.diff begins here ---
 > Index: chapter.sgml
 > ===================================================================
 > RCS file: /home/ncvs/doc/en_US.ISO8859-1/books/handbook/kernelconfig/chapter.sgml,v
 > retrieving revision 1.104
 > diff -u -r1.104 chapter.sgml
 > --- chapter.sgml	2003/03/31 17:01:34	1.104
 > +++ chapter.sgml	2003/04/17 21:55:17
 > @@ -186,6 +186,13 @@
 >        call it <filename>MYKERNEL</filename> for the purpose of this
 >        example.</para>
 >  
 > +    <note>
 > +      <para>Do <emphasis>not</emphasis> simply edit <filename>GENERIC</filename>.
 > +	If you do, it will be overwritten the next time you <link
 > +	linkend="cutting-edge">update your source tree</link>, and all
 > +	your kernel modifications will be lost.</para>
 > +    </note>
 > +
 >      <tip>
 >        <para>Storing your kernel config file directly under
 >  	<filename>/usr/src</filename> can be a bad idea.  If you are
 > --- kernelconfig.diff ends here ---
 
 The tip continues, but the diff doesn't :-)
 
 
 |      <para>Storing your kernel config file directly under
 |	<filename>/usr/src</filename> can be a bad idea.  If you are
 |	experiencing problems it can be tempting to just delete
 |	<filename>/usr/src</filename> and start again.  Five seconds after
 |	you do that you realize that you have deleted your custom kernel
 |	config file.</para>
 
 Do we need to clarify this?  If so, this should be
 added to the tip.
 
 -- 
 Marxpitn