We're currently installing Drush 6 via composer. Drush 8.0.0 will be released today. The only reason to keep using an older Drush (as far as I can tell) is to keep the PHP version below 5.4, presumably for Drupal 6 contrib. I think we should consider detecting the installed PHP version, and install Drush 8, if the version is >=5.4.5. Otherwise, fallback to Drush 6. We could also provide a debconf option to force Drush 6.

Comments

ergonlogic created an issue. See original summary.

gboudrias’s picture

+1 for detecting the PHP version. We might want to drop Drush 6 altogether in February when Drupal 6 support is dropped.

ergonlogic’s picture

Right, but I think we should hold off a bit longer than the official EOL.

gboudrias’s picture

Notes from #aegir on IRC:

  • Pretty much everyone wants to move to Drush8 by default
  • The Composer install works and allows people to move to Drush6 easily
  • We should eventually use the provided .phar but we can stick to the Composer install for now
  • PHP req's for Drush 8 will likely break some D6 contrib
  • So we need a Drush6/Drush8 debconf flag
helmo’s picture

Assigned: Unassigned » helmo
helmo’s picture

Project: Hostmaster (Aegir) » Provision

Should any one want to switch back to drush 6:
composer global require drush/drush:6.*

Debian package users can pin it to Drush 6 for future updates:
echo "aegir3-provision aegir/drush_version string 6.*" | sudo debconf-set-selections

To manually read the current setting:
debconf-show aegir3-provision

  • helmo committed 0c4527e on 7.x-3.x
    Issue #2619158: Create debconf var for Drush version, as preparation for...
  • helmo committed b9030a0 on 7.x-3.x
    Issue #2619158: Switch to Drush 8 by default
    
helmo’s picture

Project: Provision » Hostmaster (Aegir)
Status: Active » Needs review
Issue tags: +Aegir 3.3

Jenkins agrees: http://ci.aegirproject.org/view/Debian%20dev%20builds/job/D_aegir3-debia...

Anything else we need to update?
The install docs just refer to the Drush documentation ... (http://docs.aegirproject.org/en/3.x/install/#41-install-drush)

ergonlogic’s picture

Status: Needs review » Active

Drush 6 is failing to install, causing our test suite to fail. Relevant thread in Drush queue: https://github.com/drush-ops/drush/issues/1827#issuecomment-161073032

As soon as a Drush 7 phar is available, we should probably abandon Drush 6 entirely, and move away from the composer install, in favour of the phars.

helmo’s picture

Issue tags: +Aegir 3.4

As composer is giving more/new troubles the general idea is to move to using phar files asap.

E.g. #2661474: Ubuntu/debian install fails since aegir3.3

ergonlogic’s picture

Status: Active » Needs review

I've pushed commit 7d1ce9312, which should switch to using phars for Drush.

Let's see how Jenkins likes it. I'm setting to "Needs review", as I'm not certain this is the bets approach.

ergonlogic’s picture

Status: Needs review » Fixed

We're using Drush phars now for 8+, composer otherwise.

helmo’s picture

Status: Fixed » Closed (fixed)

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