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.
An obvious place for users to put global Drush options (as per the Drush docs) would be /var/aegir/.drush/drushrc.php
. Unfortunately, since we generate that file, any customizations will be lost at the next hostmaster site verify.
Let's add a locally included file for this purpose, as we do elsewhere: /var/aegir/.drush/local.drushrc.php
Comment | File | Size | Author |
---|---|---|---|
#2 | provision-allow_drush_overrides-2737785-1.patch | 1.23 KB | ergonlogic |
Comments
Comment #2
ergonlogicPatch attached, and pushed to
dev/2737785
branch.Comment #3
ergonlogicMoving to proper queue and status.
Comment #4
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedI've used /etc/drushrc.php in a few such occasions.
Moving our generated rc to aegir.drushrc.php and then on installation adding an include line to drushrc.php might be an even nicer way ... but probably more a change for 4.x
Comment #5
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedmoved dev branch to github as 'dev/2737785-local-drushrc' - https://github.com/aegir-project/provision/pull/4
And I added one commit to make '/var/aegir' a variable as we do on more places.
PS: the branch should probably have been named 'feature/2737785-local-drushrc'
Comment #7
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedcommitted
Comment #8
memtkmcc CreditAttribution: memtkmcc at Omega8.cc commented@helmo
Not sure why there is this line:
$this->data['aegir_root'] = DRUPAL_ROOT;
As far as I know,
aegir_root != DRUPAL_ROOT
?/var/aegir
already has its variable:aegir_root
, but it is neverDRUPAL_ROOT
..Maybe I have missed something?
Comment #9
colanAgreed; that looks strange.
Comment #10
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedGood catch ... Reverted ... As it was not really related to this issue I'll set the status back to fixed.