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?.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TravisCarden’s picture

Assigned: TravisCarden » Unassigned
Status: Active » Needs review
FileSize
280 bytes

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
FileSize
14.52 KB
30.18 KB
17.33 KB
30.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.

20062-before-1.png

20062-before-2.png

20062-after-1.png

20062-after-2.png

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.