I don't know whether Aegir 2.x actually supports Drupal 5, because the tests never run. Are we actually testing on a box with PHP <5.2 in Jenkins?

Either way, Drupal 5 is explicitely un-supported by Drush 5, which I believe is the new minimum recommended version for Aegir 2, no? Also, a Drush 6.x branch was recently started that requires PHP >5.3.5. It'll be the lowest Drush version to support Drupal 8, since that was just dropped from Drush 5.

I don't mean to be polemical, so we can set this to 'postponed' if we aren't ready to do this yet. But either way, it's coming, so we should prepare to be explicit about it, and clean up un-needed bits.

Comments

anarcat’s picture

anarcat commented
Title: Drop support for Drupal 5 » Drop support for Drupal 5 in 2.x
Priority: Normal » Critical

Yep, I guess this could be a reasonable API breakage of 2.x.

steven jones’s picture

steven jones commented

Drop drop drop.

helmo’s picture

helmo commented

Yes, please lets drop Drupal 5 for Aegir 2.x

anarcat’s picture

anarcat commented
Component: Drush integration » Documentation

So we seem to agree. What is there to be done here anyways? Update documentation and that's it? The Debian package already depends on Drush 5.5 or later...

anarcat’s picture

anarcat commented

Ping - what needs to be done here?

helmo’s picture

helmo commented

I've now explicitly added it to http://community.aegirproject.org/2.0-alpha1

At some point we also need to update http://community.aegirproject.org/installing/debian and community.aegirproject.org/installing/manual

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Status: Active » Fixed

I updated the installation docs, and dropped the D5 tests from Provision in f0980e0. If there's anything else, feel free to re-open this issue.

Status: Fixed » Closed (fixed)

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

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Status: Closed (fixed) » Active

Actually, there's still install_5.inc that'll need to go, at least.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Status: Active » Fixed

Fixed in bf292d8a1.

Status: Fixed » Closed (fixed)

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

  • Commit bf292d8 on dev-drupal-8, 6.x-2.x, dev-ssl-ip-allocation-refactor, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x by ergonlogic: 
    Issue #1830220: Drop support for Drupal 5.
  • Commit f0980e0 on dev-drupal-8, 6.x-2.x, dev-ssl-ip-allocation-refactor, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x by ergonlogic:
    Issue #1830220: Remove Drupal 5 tests.