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

Php7.2 #45

Merged
merged 5 commits into from
Feb 19, 2018
Merged

Php7.2 #45

merged 5 commits into from
Feb 19, 2018

Conversation

fideloper
Copy link
Contributor

Title says it all!

  1. mcrypt is replaced by sodium (mcrypt is now gone)
    • interestingly, installing php7.2-sodium creates an warning in PHP saying the module is loaded twice, as sodium is compiled into PHP from ppa:ondrej/php. Might be a bug from that repository.
  2. xdebug is not yet built for php7.2.

@mikeerickson
Copy link
Contributor

Yes, I found out the hard way last night that xDebug is not officially ready for PHP 7.2. There is however a beta release 2.6 which is purported to work with 7.2 but I didn’t have much success getting it to work.

So, downgraded back to 7.1.10 and all is good again

@nikolaynizruhin
Copy link

I guess the time has come for this PR, since xdebug 2.6.0 is out and it supports php 7.2

@fideloper
Copy link
Contributor Author

Great, thanks for the reminder!

@fideloper fideloper merged commit d8e041c into master Feb 19, 2018
@fideloper fideloper deleted the php7.2 branch February 19, 2018 13:30
MatthewBooth pushed a commit to dockfony/dockfony that referenced this pull request Sep 15, 2019
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