Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Travis: change from "trusty" to "xenial" #7

Merged
merged 1 commit into from
Jul 30, 2020

Conversation

jrfnl
Copy link
Collaborator

@jrfnl jrfnl commented Jul 20, 2020

As the "trusty" environment is no longer officially supported by Travis, they decided in their wisdom to silently stop updating the PHP "nightly" image, which makes it next to useless as the last image apparently is from January....

This updates the Travis config to:

  • Use the xenial distro, which at this time is the default.
  • Sets the distro for low PHP versions explicitly to trusty.
  • Makes the expected OS explicit (linux).

Note: the PHP 8 build will currently fail. Fixes for that will be pulled separately.

As the "trusty" environment is no longer officially supported by Travis, they decided in their wisdom to silently stop updating the PHP "nightly" image, which makes it next to useless as the last image apparently is from January....

This updates the Travis config to:
* Use the `xenial` distro, which at this time is the default.
* Sets the distro for low PHP versions explicitly to `trusty`.
* Makes the expected OS explicit (linux).
* The `xenial` distro is quite bare, so `ant` is not available by default. Adding this as an `addon` fixes that.
@jrfnl
Copy link
Collaborator Author

jrfnl commented Jul 21, 2020

Note: the build didn't error, it just stalled, so someone who has got the rights to do so should restart the stalled build.

@grogy
Copy link
Member

grogy commented Jul 30, 2020

Great! It works perfect. Thank you

@grogy grogy merged commit 666889f into php-parallel-lint:master Jul 30, 2020
@jrfnl jrfnl deleted the feature/update-travis-env_ branch July 30, 2020 13:23
@jrfnl jrfnl added this to the 1.0.0 milestone Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants