These days in Drupal core, I'm running into PHP major version incompatibilities almost on a daily basis.

To figure out why a patch failed, it would be very helpful if the test review log would contain the exact PHP version of the testbot.

This is also a preparative step for #2135199: Provide php 5.4 testing on testbots for D8 code without breaking everything else

CommentFileSizeAuthor
project_issue_file_review.php-version.0.patch538 bytessun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jthorson’s picture

jthorson CreditAttribution: jthorson commented
Status: Needs review » Fixed

Committed to both 6.x-2.x and 6.x-3.x branches.

Will be deployed with the next release.

Thanks!

jthorson’s picture

jthorson CreditAttribution: jthorson commented
Status: Fixed » Needs work

Ooops ... this threw a php fatal related to the $this->log() command within the this->exec() function.

Back to needs work (though I should be able to get to it soon).

YesCT’s picture

YesCT CreditAttribution: YesCT commented

#2296751: Have test results log report the actual php version it used
And
#2400237: Testbot should report exact PHP version.
closed as duplicates

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Project: Project Issue File Review » DrupalCI: Results (Modernizing Testbot Initiative)
Version: 6.x-2.x-dev »
Component: Code » User interface
Status: Needs work » Active
Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
CreditAttribution: Mixologic at Drupal Association commented
Status: Active » Fixed
Issue tags: -PHP 5.4

This now exists and has been pushed to production. Thanks @jthorson

Status: Fixed » Closed (fixed)

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