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.
While the making of #2482549: Ignore node_module folder in core to use Drupal with npm/grunt/nodejs, a wrong permission has been committed on the file default.settings.php by mistake.
This patch will restore the original permission.
Comment | File | Size | Author |
---|---|---|---|
#2 | 3035772.patch | 117 bytes | Pol |
Comments
Comment #2
PolComment #5
PolAdding credits to @xlin.
Comment #6
alexpott@Pol I wonder if the D7 maintainers should fork https://github.com/alexpott/d8githooks and use a pre commit hook to check this. It's hard to pick up.
Comment #7
PolVery good idea, going to see how to use it for D7.
Thanks!
Comment #9
PolFixed.
Comment #11
xlin CreditAttribution: xlin as a volunteer commented@pol Same issue since 7.64, default.settings.php permission is incorrect again.
Comment #12
PolHi,
Could you point to the commit where it has been re-introduced ?
Thx
Comment #13
gregglesReopening to track the work.
Comment #14
tobiasbIn git all is fine only the download contains a default.settings.php with an execute bit.
Comment #15
drummThe recent security releases 7.65, 7.66, and 7.67 do not include other fixes. They only contain the security fix to make quick upgrades more straightforward. They are branched from the previous release, then merged back into dev.
The last non-security release of 7.x was 7.64, which predates this fix. So it has not made it into a tagged release yet. The dev release looks good.