On Wed, Jul 11, 2012 at 11:26 AM, Ed Heron <Ed@xxxxxxxxxxxxx> wrote:
> On Wed, 2012-07-11 at 10:42 -0400, Brian Mathis wrote:
>> Requesting access to edit page TipsAndTricks/ApacheVhostDir
>>
>> Looking to make some small edits for clarity.
>>
>> ❧ Brian Mathis
>
> Yay, somebody read it!
>
> What are you suggesting?
The use of "mv -v ...{,_}" is too clever for this kind of educational
document, and should be changed to spell out the full "mv" command. I
get what you're doing there, but the purpose of the document is not to
teach clever uses of bash, it's to make it obvious to people that
you're renaming the file. It will trip up the flow of reading for all
but the most knowledgeable users, and users who don't understand it
will be totally lost.
In most documents and scripts, I usually spell out the short form
options as well, such as using "--verbose". Short forms save you
typing, but documentation should not trip people up if they don't know
what the option means.
Also, I find the use of "_" to be obtuse and highly error prone if one
were to actually run a server that way. It's far more obvious to use
"disabled", which makes it very clear that those items are disabled.
It may work for you but only because that's a convention you came up
with so you're used to it, but we're not in dos 8.3 days with
filenames, so why not be more descriptive?
In section 6.4, is there a reason not to make a "vhosts.conf" file
that contains the "Include" in the in the conf.d/ directory, instead
of appending to the httpd.conf, or do you run into ordering issues
there? I try to avoid changing the distro files if possible.
❧ Brian Mathis
_______________________________________________
CentOS-docs mailing list
CentOS-docs@xxxxxxxxxx
http://lists.centos.org/mailman/listinfo/centos-docs
