Skip to content

Add Redis to 7-fpm image.#31

Merged
zaporylie merged 1 commit intodrupal-docker:masterfrom
slashrsm:redis
May 25, 2016
Merged

Add Redis to 7-fpm image.#31
zaporylie merged 1 commit intodrupal-docker:masterfrom
slashrsm:redis

Conversation

@slashrsm
Copy link
Copy Markdown
Contributor

Adding Redis to 7-fpm. I am happy to add it to other images if needed.

@slashrsm
Copy link
Copy Markdown
Contributor Author

Related to #28

@zaporylie
Copy link
Copy Markdown
Member

pecl/redis requires PHP (version >= 5.2.0, version <= 6.0.0, excluded versions: 6.0.0), installed version is 7.0.6

@slashrsm slashrsm force-pushed the redis branch 2 times, most recently from a04d421 to 4fc4872 Compare May 18, 2016 16:20
@slashrsm
Copy link
Copy Markdown
Contributor Author

Redis extension needs to be build manually for PHP 7 it seems. Did that and added it to other php versions too. Also rebased against master. Let's see what Travis thinks about it now.

@zaporylie
Copy link
Copy Markdown
Member

I decided to merge it into master however I am still a little bit concerned about maintaining additional extension, especially with no stable release. I would like to add to README file information about all the features supported by php image (i.e. redis) and information about maintenance status (i.e. redis - experimental).

I believe we should use pecl for phpredis/php7 ASAP (according to phpredis issue queue extension should be released soon™).

@zaporylie zaporylie merged commit da27e17 into drupal-docker:master May 25, 2016
@slashrsm
Copy link
Copy Markdown
Contributor Author

I decided to merge it into master however I am still a little bit concerned about maintaining additional extension, especially with no stable release. I would like to add to README file information about all the features supported by php image (i.e. redis) and information about maintenance status (i.e. redis - experimental).

Sure. Let's do that. If I find any time in the next few days I'll start working on this and create a pull.

I believe we should use pecl for phpredis/php7 ASAP (according to phpredis issue queue extension should be released soon™).

Agreed.

Thank you for merging!

@zaporylie
Copy link
Copy Markdown
Member

Sure. Let's do that. If I find any time in the next few days I'll start working on this and create a pull.

Great!

Thank you for merging!

Thank you!

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.

2 participants