Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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
Comment #2
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented+1 for detecting the PHP version. We might want to drop Drush 6 altogether in February when Drupal 6 support is dropped.
Comment #3
ergonlogicRight, but I think we should hold off a bit longer than the official EOL.
Comment #4
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commentedNotes from #aegir on IRC:
Comment #5
helmo CreditAttribution: helmo at Initfour websolutions commentedComment #6
helmo CreditAttribution: helmo at Initfour websolutions commentedShould 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
Comment #8
helmo CreditAttribution: helmo at Initfour websolutions commentedJenkins 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)
Comment #9
ergonlogicDrush 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.
Comment #10
helmo CreditAttribution: helmo at Initfour websolutions commentedAs 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
Comment #11
ergonlogicI'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.
Comment #12
ergonlogicWe're using Drush phars now for 8+, composer otherwise.
Comment #13
helmo CreditAttribution: helmo at Initfour websolutions commentedOne follow up is still needed: #2667194: Update upgrade.sh.txt to use phars for drush