en/handbook/kernelconfig: proposed corrections

[Taras Korenko]

On Wed, Feb 05, 2014 at 12:37:48AM +0400, Sergey Kandaurov wrote:
> On Tue, Feb 04, 2014 at 12:16:04PM +0200, Taras Korenko wrote:
> > ...
> >   So, could anyone review that diff?
> 
> Hi, see my 2c inline.
> 
> > Index: en_US.ISO8859-1/books/handbook/kernelconfig/chapter.xml
> > ===================================================================
> > --- en_US.ISO8859-1/books/handbook/kernelconfig/chapter.xml	(revision 43745)
> > +++ en_US.ISO8859-1/books/handbook/kernelconfig/chapter.xml	(working copy)
> > ...
> > @@ -226,7 +226,7 @@
> >  ath_hal(4)               - Atheros Hardware Access Layer (HAL)</programlisting>
> >  
> >      <para>Once the hardware inventory list is created, refer to it
> > -      to ensure that installed hardware is not removed as you edit the custom
> > +      to ensure that drivers for installed hardware are not removed as you edit the custom
> Or rather "drivers OR installed hardware"? :) Both may be removed I guess.
I guess user's task at this moment is to create a subset from all
available drivers, noone is to add/pull_out NICs, videocards, etc...
> 
> >        kernel configuration file.</para>
> >    </sect1>
> >  
> > @@ -250,11 +250,7 @@
> >        <para>If <filename>/usr/src/</filename> does
> >  	not exist or it is empty, source has not been installed.
> >  	Source can be installed using
> > -	<application>svn</application>, which is described in <xref linkend="svn"/>, or by installing the
> > -	<literal>src</literal> distribution using &man.sysinstall.8;.  This
> > -	distribution can be selected by navigating to the 
> > -	<literal>Configuration</literal> and then to the
> > -	<literal>Distributions</literal> menu within &man.sysinstall.8;.</para>
> > +	<application>svn</application>, which is described in <xref linkend="svn"/>.</para>
> You removed reference to sysinstall, but it is still in use in 9.x and 8.x
> (well ok, only in 8.x but it also exists in 9.x).
> If you want to touch this place, I would suggest to specify &os; version
> this is applicable for and briefly mention bsdinstall with referring to
> <xref linked="bsdinstall-components"/>.  It would be fine with simply
> removing sysinstall part, but this leaves the reader without options.

Well,
  OS 8.X: sysinstall allows us to install sources.
  OS 9.X: according to our hb, sysinstall is deprecated.  Bsdinstall
cannot install sources because it does not have such a post-install
procedure.
  OS 10.X: we've svnlite (but cannot refer to it's man page).

So, we've to provide detailed description of all available options, or
keep just one.
> ...

-- 
  WBR, Taras Korenko