Bug 140703

Summary: [patch] add xorg-minimal as alternative for X11 Distribution installation
Product: Documentation Reporter: Jason <jhelfman>
Component: Books & ArticlesAssignee: freebsd-doc (Nobody) <doc>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Jason 2009-11-19 18:50:02 UTC 
add xorg-minimal as alternative for X11 Distribution installation

-suggestion per maintainer of port

Attempted to fix this particular portion of document per standards I believe I met.

Fix: Patch attached with submission follows:
Comment 1 Marc Fonvieille freebsd_committer freebsd_triage 2009-11-19 19:26:20 UTC 
On Thu, Nov 19, 2009 at 06:46:35PM +0000, Jason Helfman wrote:
> 
> >Description:
> add xorg-minimal as alternative for X11 Distribution installation
> 
> -suggestion per maintainer of port
> 
> Attempted to fix this particular portion of document per standards I believe I met.

[...]

Hello,

Most of time we avoid to mix content and whitespace changes.

> 
> --- en_US.ISO8859-1/books/handbook/x11/chapter.sgml.orig	2009-11-19 10:33:01.000000000 -0800
> +++ en_US.ISO8859-1/books/handbook/x11/chapter.sgml	2009-11-19 10:42:19.000000000 -0800
> @@ -339,11 +339,18 @@
>  
>      <screen>&prompt.root; <userinput>pkg_add -r xorg</userinput></screen>
>  
> -    <note><para>The examples above will install the complete
> +    <note>
> +      <para>The examples above will install the complete
>        X11 distribution including the
>        servers, clients, fonts etc.  Separate packages and ports of X11
> -      are also
> -      available.</para></note>
> +      are also available.</para>

All above are whitespace/wrapping/layout changes.

> +
> +      <para>To install an minimal X11 distribution you can 
> +      alternatively install <filename role="package">x11/xorg-minimal
> +      </filename></para>.

	the closing </filename> should directly follow x11/xorg-minimal,
	i.e., <filename role="package">x11/xorg-minimal</filename>.
	You can break the tags like this:

	blahblahblah blah blahblahblahblahblahblah <filename
	role="package">x11/xorg-minimal</filename>

	but not like:

	blahblahblah blahblahblah <filename role="package">x11/xorg-minimal
	</filename>


-- 
Marc
Comment 2 Jason 2009-11-19 19:28:14 UTC 
That should be "a minimal..." not "an minimal..."

-jgh
Comment 3 Benedict Reuschling freebsd_committer freebsd_triage 2010-07-11 17:17:40 UTC 
State Changed
From-To: open->closed

A modified version of your original patch was committed.  
Thanks for your contribution. PR closed.
Comment 4 dfilter service freebsd_committer freebsd_triage 2010-07-11 17:18:46 UTC 
bcr         2010-07-11 16:15:45 UTC

  FreeBSD doc repository

  Modified files:
    en_US.ISO8859-1/books/handbook/x11 chapter.sgml 
  Log:
  Add a short paragraph about the x11/xorg-minimal package as
  an alternative to the complete X11 distribution.
  
  PR:             docs/140703
  Submitted by:   Jason Helfman (jhelfman at e dash e dot com)
  OK'ed by:       blackend@
  
  Revision  Changes    Path
  1.202     +5 -1      doc/en_US.ISO8859-1/books/handbook/x11/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"