Bug 207344

Summary: Grammar Fixes
Product: Documentation Reporter: harrij15
Component: Books & ArticlesAssignee: Warren Block <wblock>
Status: Closed FIXED    
Severity: Affects Only Me CC: wblock
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Fixed errors in a paragraph of Jails none

Description harrij15 2016-02-19 18:04:25 UTC 
Replaced commas with parentheses for clarity in the Jails section.
Also, changed chrooted to "chroot-ed" in the same paragraph.
Comment 1 harrij15 2016-02-19 18:12:34 UTC 
Created attachment 167190 [details]
Fixed errors in a paragraph of Jails
Comment 2 Warren Block freebsd_committer freebsd_triage 2016-02-19 21:13:44 UTC 
The diff deletes and re-adds that whole file.  Is it a line ending problem (CRLF rather than just LF), or indentation changed from tabs to spaces?
Comment 3 harrij15 2016-02-20 04:12:17 UTC 
The line containing "which is used to change the root directory of a set of processes" was changed, so that the commas are now parentheses. Also, two sentences below that sentence, the word chrooted was changed to "chroot-ed."
Comment 4 Warren Block freebsd_committer freebsd_triage 2016-02-20 04:35:54 UTC 
Understood, I'm just saying the diff is malformed, showing that the entire file changed.

That sentence has problems which our fallible human understanding of parentheses might not be able to cure.  Put another way, there are at least two sentences worth of stuff crammed into that single sentence.  So I split it up.  This also avoids asides which divert the reader's attention.

As far as "chrooted", there are several forms, each of which is uglier than the one before it, but of course only if you line them up that way.  People sometimes jam an apostrophe in there.  I personally find "chrooted" less ugly than the alternate forms.  Here, it is not talking about the chroot(8) command, which could be marked up in DocBook as "<command>chroot</command>ed".  It appears to me to be using it as an adjective, describing the environment created by chroot(8).  That sentence does need another comma, though.
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-02-20 04:38:10 UTC 
A commit references this bug:

Author: wblock
Date: Sat Feb 20 04:38:05 UTC 2016
New revision: 48220
URL: https://svnweb.freebsd.org/changeset/doc/48220

Log:
  Fixes for grammar and punctuation.

  PR:		207344
  Submitted by:	harrij15@rpi.edu

Changes:
  head/en_US.ISO8859-1/books/handbook/jails/chapter.xml
Comment 6 Warren Block freebsd_committer freebsd_triage 2016-02-20 04:44:49 UTC 
Thank you for submitting this, changes committed!
Comment 7 harrij15 2016-02-20 04:46:13 UTC 
Oh, I apologize for any inconveniences regarding the diff, and the changes you made make a lot more sense. Thanks for committing the changes!