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: Morbus Iff Status: patch 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. Morbus Iff Previous comments: ------------------------------------------------------------------------ March 28, 2005 - 07: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.

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: Chris Johnson Status: patch The correct English capitalization is the same as Dutch -- if that's what we are talking about here. Chris Johnson Previous comments: ------------------------------------------------------------------------ March 28, 2005 - 06: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 - 09: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 - 11: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

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.

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: Dries -Status: patch +Status: fixed Committed to HEAD. Thanks. Dries 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. ------------------------------------------------------------------------ March 31, 2005 - 23:31 : Uwe Hermann Attachment: http://drupal.org/files/issues/INSTALL.txt_0.patch (1.05 KB) New patch, please review. ------------------------------------------------------------------------ March 31, 2005 - 23:40 : Morbus Iff +1 from me.