Tests are failing current because we use latest master branch of Drush, which needs PHP 5.5:

https://travis-ci.org/panopoly/panopoly/jobs/116469779

Pin to Drush 8!

CommentFileSizeAuthor
#2 panopoly-drush-7-2688644.patch446 bytesdsnopek
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Active » Needs review
FileSize
panopoly-drush-7-2688644.patch446 bytes

Here's a first go!

EDIT: Here's the Travis build: https://travis-ci.org/panopoly/panopoly/builds/116473119

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented

Gah! Tested that wrong. Here's a better go:

https://travis-ci.org/dsnopek/panopoly/builds/116474131

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Needs review » Needs work

Blergh. And that doesn't work because it's overwritten by the drupal_ti "environment":

https://github.com/LionsAd/drupal_ti/blob/59fa84993afebdd69d203088a4ebd0...

This probably should just be fixed in drupal_ti...

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Title: Pin to Drush 7 so we can keep testing with PHP 5.4 » Pin to Drush 8 so we can keep testing with PHP 5.4
Issue summary: View changes

Attempting to fix in drupal_ti:

https://github.com/LionsAd/drupal_ti/pull/71

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Needs work » Fixed

Fixed upstream in drupal_ti! Tests now appear to be passing. :-)

Status: Fixed » Closed (fixed)

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