[patch] content changes to X11 chapter

Dru Lavigne dru.lavigne at att.net
Wed Jan 16 14:39:54 UTC 2013


--- On Sun, 1/13/13, Marc Fonvieille <blackend at freebsd.org> wrote:

> From: Marc Fonvieille <blackend at freebsd.org>
> Subject: Re: [patch] content changes to X11 chapter
> To: "Dru Lavigne" <dru.lavigne at att.net>
> Cc: freebsd-doc at freebsd.org
> Date: Sunday, January 13, 2013, 5:28 PM
> On Sun, Jan 13, 2013 at 08:58:50AM
> -0800, Dru Lavigne wrote:
> > 
> > 
> > > >> On Fri, Jan 11, 2013 at 02:31:52PM -0800,
> Dru
> > > Lavigne wrote:
> > > >>> Attached is the svn diff for the X11
> chapter.
> > > This
> > > >> represents a
> > > >>> complete content update of this
> chapter.
> > > >>> 
> > > >>> Igor is mostly happy. I think the
> remaining
> > > indent
> > > >> errors are false
> > > >>> warnings as they only occur when an
> inline tag
> > > is in
> > > >> the previous
> > > >>> <para>.
> > > >>> 
> > > >>> I did not verify the widescreen
> settings. I
> > > think the
> > > >> i810 section is
> > > >>> correct. The KDM section needs more
> content.
> > > Other than
> > > >> that, it is a
> > > >>> decent first pass at bringing this
> chapter into
> > > the
> > > >> 21st century.
> > > >>> 
> > > >> 
> > > >> Some thoughts:
> > > >> 
> > > >> - the whole diff will be difficult to
> read for a
> > > translator: 
> > > >> section moves, whitespace & content
> changes,
> > > etc. It will be
> > > >> important to commit in separate parts:
> for example:
> > > 1st section 
> > > >> moves/removals, 2nd addition &
> content changes
> > > and 3rd 
> > > >> whitespace/indentation changes;
> > > >> 
> > > >> - you removed the prerequisite of knowing
> how to
> > > use ports, I
> > > >> think it's important to know how to get a
> ports
> > > tree before using
> > > >> the ports;
> > > >> 
> > > >> - you removed the way to get a localized
> keyboard
> > > for people not
> > > >> using a desktop manager.  Well this
> > > configuration is not easy to
> > > >> guess for a newcomer.  Maybe it can be
> kept
> > > somewhere in the 
> > > >> chapter.
> > > >> 
> > > >> - What is # in Crtl+Alt+F#?


Attached is a cleaned up diff. I've re-added the pre-requisite and fixed the Crtl+Alt+F#. Marc, can you double-check that the localization text you are looking for is incorporated into 6.8.2? If that is not it, please quote me some of the missing text so I can re-add it.

Cheers,

Dru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: x11_take2.diff
Type: text/x-patch
Size: 101843 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-doc/attachments/20130116/f3b82101/attachment-0001.bin>


More information about the freebsd-doc mailing list