Skip to content

Conversation

@oohnoitz
Copy link
Contributor

@oohnoitz oohnoitz commented Dec 15, 2016

This adds the following:

  • php 7.1
  • sphinxsearch 2.3 (rel23)

Note: sphinxsearch-rel23 branch doesn't have any builds at the moment. Do not merge for now.
https://launchpad.net/~builds/+archive/ubuntu/sphinxsearch-rel23

@oohnoitz oohnoitz self-assigned this Dec 15, 2016
@alpha0010
Copy link
Contributor

Probably need to remove PHP 5.3 from the matrix. travis-ci/travis-ci/issues/7163

@oohnoitz
Copy link
Contributor Author

oohnoitz commented Sep 6, 2017

Yeah. There's still some compatibility issues with PHPUnit and type checking on some versions.

@oohnoitz
Copy link
Contributor Author

oohnoitz commented Nov 3, 2017

Going to take another look at this during the weekend. Think we'll drop support for PHP 5.3 since it reached EOL over 3 years ago and a cause of pain for the PDO connector.

@alpha0010
Copy link
Contributor

If we want to keep PHP 5.3, try #125.

@oohnoitz oohnoitz merged commit ca6e683 into master Nov 7, 2017
@oohnoitz
Copy link
Contributor Author

oohnoitz commented Nov 7, 2017

That is an option but I rather just keep our test matrix as simple as possible for now. We can revisit this decision later if necessary.

@oohnoitz oohnoitz deleted the f-travis-ci-sphinx-rel23-php-71 branch November 7, 2017 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants