WordPress.org

Make WordPress Core

Opened 22 months ago

Last modified 6 months ago

#42811 new defect (bug)

Run a Travis job with the MySQL Extension

Reported by: dd32 Owned by:
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch needs-refresh
Focuses: Cc:

Description (last modified by dd32)

I just noticed that the PHP 5.2 job is running with MySQLi (which I didn't even realise was possible).

That ultimately means we have no automated testing which is testing the mysql_* functions in $wpdb which is used for PHP 5.2~5.4 by default.

The attached patch uses WP_USE_EXT_MYSQL to force it to use the MySQL extension for the PHP 5.2 job, I've tested this on my own travis account and it passes as expected, with the only change being an additional skipped test (which requires mysqli).

Attachments (1)

42811.diff (1.5 KB) - added by dd32 22 months ago.

Download all attachments as: .zip

Change History (9)

@dd32
22 months ago

#2 @pento
22 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 5.0

What an exciting time to be alive!

#3 @dd32
22 months ago

  • Description modified (diff)

Just noticed I incorrectly referenced mysqli_* functions instead of mysql_* functions :)

#4 @dd32
20 months ago

I got to thinking - now that PHP 5.2 could legitimately be running MySQLi in production, we probably shouldn't take the route in 42811.diff directly.

Maybe this means we need another job dedicated to MySQL - although it doesn't need to be PHP 5.2, perhaps it could be a 5.6 job?
If we were to limit PHP 5.2 jobs to MySQL-only we'd miss out on the regular testing of the least-feature-complete version of MySQLi out there.

#5 @jorbin
11 months ago

  • Milestone changed from 5.0 to 5.1

#6 @pento
8 months ago

  • Keywords needs-refresh added; commit removed
  • Milestone changed from 5.1 to 5.2

With WordPress' PHP version bump coming up, let's just add an extra PHP 5.6 job that tests with ext/mysql.

#7 @jorbin
6 months ago

  • Milestone changed from 5.2 to 5.3

Bumping this from 5.2 due to lack of progress but as it is related to a test config and not something that directly affects the code we release, I'm happy to bring it back if there is progress.

#8 @johnbillion
6 months ago

  • Summary changed from Run the PHP 5.2 Travis job with the MySQL Extension to Run a Travis job with the MySQL Extension
Note: See TracTickets for help on using tickets.