Along the same lines as #1841896: Fix various file permissions, some more files have crept into the codebase with unnecessary executable bits (0755 perms). A patch to fix will follow. See also #1891270: [discussion] How can we prevent "invisible" regressions?.

#4 20062-before-1.png30.32 KBandymartha
#4 20062-before-2.png17.33 KBandymartha
#4 20062-after-1.png30.18 KBandymartha
#4 20062-after-2.png14.52 KBandymartha
#1 drupal-more-file-perms-2006258-1.patch280 bytesTravisCarden
PASSED: [[SimpleTest]]: [MySQL] 56,573 pass(es).
[ View ]


TravisCarden’s picture

Assigned:TravisCarden» Unassigned
Status:Active» Needs review
new280 bytes
PASSED: [[SimpleTest]]: [MySQL] 56,573 pass(es).
[ View ]

Status:Needs review» Needs work
Issue tags:-Quick fix, -Novice

The last submitted patch, drupal-more-file-perms-2006258-1.patch, failed testing.

TravisCarden’s picture

Status:Needs work» Needs review
Issue tags:+Quick fix, +Novice
andymartha’s picture

Status:Needs review» Reviewed & tested by the community
new14.52 KB
new30.18 KB
new17.33 KB
new30.32 KB

I can confirm that after applying patch drupal-more-file-perms-2006258-1.patch to a fresh installation of Drupal 8.x-dev by TravisCarden in #1, the files mentioned change from being executable to read and write only. See screenshots.


alexpott’s picture

Status:Reviewed & tested by the community» Fixed

Committed 8f85c84 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.