[drupal-devel] [bug] Minor fixes in INSTALL.txt
Uwe Hermann
drupal-devel at drupal.org
Thu Mar 31 21:31:59 UTC 2005
Issue status update for http://drupal.org/node/19588
Project: Drupal
Version: cvs
Component: base system
Category: bug reports
Priority: minor
Assigned to: Anonymous
Reported by: Uwe Hermann
Updated by: Uwe Hermann
Status: patch
Attachment: http://drupal.org/files/issues/INSTALL.txt_0.patch (1.05 KB)
New patch, please review.
Uwe Hermann
Previous comments:
------------------------------------------------------------------------
March 28, 2005 - 14:24 : Uwe Hermann
Attachment: http://drupal.org/files/issues/INSTALL.txt.patch (1.44 KB)
Hi,
here's a tiny patch which fixes some issues in the INSTALL.txt file.
------------------------------------------------------------------------
March 28, 2005 - 17:29 : Morbus Iff
I'm fine with change #1 and #3, but leery about #2. In a suEXEC
environment, the "webserver process" may have different permissions
than the "Drupal server process" (or, more accurately, the User/Group
of the VirtualHost running the Drupal code). With that said, it does
"read" better to me. So, I guess I'm fine with it, having stated my
misgivings.
------------------------------------------------------------------------
March 28, 2005 - 19:24 : Dries
I know that in Dutch, we write "NOTE: for" and not "NOTE: For". I don't
recall us doing that elsewhere either. As such, I'm not convinced that
chunk #1 is OK. I
------------------------------------------------------------------------
March 28, 2005 - 20:20 : Chris Johnson
The correct English capitalization is the same as Dutch -- if that's
what we are talking about here.
------------------------------------------------------------------------
March 28, 2005 - 20:38 : Anonymous
I'm fine with a -1 on #1 too.
More information about the drupal-devel
mailing list