On Wed, Jul 11, 2012 at 11:26 AM, Ed Heron <Ed at heron-ent.com> 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