Drupal 10, the latest version of the open-source digital experience platform with even more features, is here.(this pertains to the "new" branch)
I'm building a custom puppet script that installs Drush 6 via the Puppet Drush module (using the Drush::Git::Drush class), and then installing Aegir 2 via this module.
This module, however, attempts to install Drush without first checking to see if it's already installed.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | puppet-aegir_drush_exists-2064603-3.patch | 872 bytes | m.stenta |
| #1 | puppet-aegir_drush_exists-2064603-1.patch | 969 bytes | m.stenta |
Comments
Comment #1
m.stentaTalked to @ergonlogic in IRC, and put together a patch for it.
Comment #2
ergonlogicFirst off, thanks again!
Why did you put the file resource in the conditional block? This is essentially there to remove files (since deprecated) from the old puppet-drush module.
Comment #3
m.stentaAh, gotcha. I didn't know that's what those were for. Updated patch attached (I also included your description as a comment for others to help understand those lines).
Comment #4
ergonlogicfixed in 80661c5. Thanks again!
Comment #5
m.stentaDang! I just realized that this doesn't work, because the lines that delete those deprecated files send a notification to "Exec['drush_update_apt']", which doesn't exist in this context (where Drush is already installed and we're not including drush here).
Should we just remove the
notify => Exec['drush_update_apt']line? Is it important to have?Comment #6
ergonlogicfixed in bf54444 by changing to:
notify => Exec['aegir_update_apt']