[buddypress-trac] [BuddyPress Trac] #5708: Broken Travis CI PHP 5.2 tests and improved Travis CI performance

[buddypress-trac]

#5708: Broken Travis CI PHP 5.2 tests and improved Travis CI performance
--------------------------+---------------------
 Reporter:  netweb        |       Owner:
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  2.1
Component:  Core          |     Version:
 Severity:  normal        |  Resolution:  fixed
 Keywords:  has-patch     |
--------------------------+---------------------
Changes (by boonebgorges):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 In the series of changesets above, I've taken the following from netweb's
 original patch:

 - moved to Git checkouts of WP
 - set fast_finish: true
 - reenabled `WP_DEBUG` on PHP 5.5 tests when running WP > 3.7
 - moved most environment setup to `before_install`

 I was able to solve the PHP 5.2 issue without resorting to changes in our
 PHPUnit bootstrap or explicit references to Travis in BP. Specifically,
 the problem was related to the use of the `export` command when setting
 environment variables (it appears that PHP 5.2 was not running in the same
 shell session as the Travis setup script - maybe it's sandboxed or
 something).

 Regarding HHVM:

 > there has been work to get WordPress running on HHVM and fully expect
 this to be a supported PHP version in the future.

 You may be right about this, but let's cross this bridge when we come to
 it. I'd rather not muck about in BP and waste Travis resources running
 builds that we're not planning to support. When WP announces full support
 for HHVM, let's reassess.

 Thanks again for the initial patch, netweb.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5708#comment:13>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac