Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
here's a tiny patch which fixes some issues in the INSTALL.txt file.
Comment | File | Size | Author |
---|---|---|---|
#5 | INSTALL.txt_0.patch | 1.05 KB | Uwe Hermann |
INSTALL.txt.patch | 1.44 KB | Uwe Hermann | |
Comments
Comment #1
Morbus IffI'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.
Comment #2
Dries CreditAttribution: Dries commentedI 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
Comment #3
Chris Johnson CreditAttribution: Chris Johnson commentedThe correct English capitalization is the same as Dutch -- if that's what we are talking about here.
Comment #4
(not verified) CreditAttribution: commentedI'm fine with a -1 on #1 too.
Comment #5
Uwe Hermann CreditAttribution: Uwe Hermann commentedNew patch, please review.
Comment #6
Morbus Iff+1 from me.
Comment #7
Dries CreditAttribution: Dries commentedCommitted to HEAD. Thanks.
Comment #8
(not verified) CreditAttribution: commented