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

Fixes #501: Default Drupal VM to PHP 5.6. #502

Merged
merged 1 commit into from
Oct 6, 2016
Merged

Fixes #501: Default Drupal VM to PHP 5.6. #502

merged 1 commit into from
Oct 6, 2016

Conversation

geerlingguy
Copy link
Contributor

See #501

@danepowell
Copy link
Contributor

This broke VM initialization, because the package php-bz2 doesn't exist for Trusty (14.04). See #521

@geerlingguy
Copy link
Contributor Author

@danepowell - #503 fixes this, but is waiting in a conga line of selenium/phantom-related PRs.

We could hotfix right now by setting bz2 as a php_pecl_extension instead (to make sure the 8.x HEAD branch still builds the VM until/unless selenium replaces Phantom), but we would also need to remove the bcmath installed_extra (because it also doesn't actually exist for PHP 5.6 :).

cc @grasmash — basically, we need to either get #503 merged (after phantom is no longer a requirement), or we can make a separate PR to get things building in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants