Hi,

here's a tiny patch which fixes some issues in the INSTALL.txt file.

CommentFileSizeAuthor
#5 INSTALL.txt_0.patch1.05 KBAnonymous (not verified)
INSTALL.txt.patch1.44 KBAnonymous (not verified)

Comments

morbus iff’s picture

morbus iff
they/them
commented

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.

dries’s picture

dries commented

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

Chris Johnson’s picture

Chris Johnson commented

The correct English capitalization is the same as Dutch -- if that's what we are talking about here.

Anonymous’s picture

(not verified) commented

I'm fine with a -1 on #1 too.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new INSTALL.txt_0.patch1.05 KB

New patch, please review.

morbus iff’s picture

morbus iff
they/them
commented

+1 from me.

dries’s picture

dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) commented