patch for bug #109719
Phil Bull
philbull at gmail.com
Wed Jan 23 11:48:23 UTC 2008
Hi Andrew,
On Tue, 2008-01-22 at 14:47 -0600, Andrew Stabeno wrote:
> This patch fixes bug #109719, by adding a chart and organizing the
> original authors thoughts a little better. I also removed a linkend
> that was broken, that I caught with the validate.sh tool. Please
> review and commit if no changes are need.
Thanks for the patch, it looks good! I found a few minor issues:
* "each of the thee sets" => "each of the three sets"
* "permissions and how they effect files" => "permissions and
how they affect files"
* "directories can be entered" - this aspect of file permissions
must seem a little strange to Windows users, so they might not
immediately know what is meant by "entering" a directory. It
might be worth being a bit more explicit here, e.g. "the
contents of the directory can be viewed".
* "to update this index run the command
<application>updatedb</application>." - while you do mention
that the command needs administrative privileges, it's probably
best to put the full "sudo updatedb" instead of just "updatedb",
as there's no guarantee that a user will read the following
sentences after finding the name of the command.
* It could also be worth noting that updatedb takes quite a while
to index the whole filesystem, and doesn't give any output.
Thanks again,
Phil
--
Phil Bull
http://www.launchpad.net/people/philbull
More information about the ubuntu-doc
mailing list