We already allow sites/all/platform.settings.php, but I think this is a little too "Aegir-y".

If we can use "sites/all/settings.php" it will be easier to convince existing teams to make the site aegir ready.

Many many folks use sites/default/settings.php for global configuration and commit that to the repo. I want to encourage sites/all/settings.php for this type of requirement.

Comments

Jon Pugh created an issue. See original summary.

  • Jon Pugh committed 74e8486 on 3055463-more-includes 
    Issue #3055463: Add sites/all/settings.php to the include list.
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
Status: Active » Needs review
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
Issue tags: +devshop patches
helmo’s picture

helmo commented
Status: Needs review » Reviewed & tested by the community

+1 for supporting mainstream :)

We might even add de deprecation warning on the platform.settings.php lines, then 4.x can drop it. Less is more

jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
Status: Reviewed & tested by the community » Fixed

  • Jon Pugh committed 74e8486 on 7.x-3.x 
    Issue #3055463: Add sites/all/settings.php to the include list.

Status: Fixed » Closed (fixed)

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