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

Allow different web server than nginx #24

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

webflo
Copy link
Contributor

@webflo webflo commented Apr 28, 2013

I think it makes sense to decouple the webserver configuration from php::fpm::fastcgi and php::fpm::service for more flexibility.

I am not sure if this is a good approach to do this. I relatively new to puppet.

Thanks!

webflo added 5 commits April 28, 2013 23:11
* origin/master:
  Bump version to 0.5.6, fixes more zlib issues
  Detect broken versions of PHP and fail existence check
  libtool patch isn’t required for zlib 1.2.8.
  Require zlib 1.2.8 in manifest.
  Update zlib to 1.2.8.
  Bump cardboard dependency
  Bump version
  Add 5.3.24, 5.4.12, 5.4.13, 5.4.14.
  Lint mcrypt manifest
  Add mcrypt extension, using php_source provider
  Add php extension provider which can build extensions bundled in the php source
@adnasa
Copy link

adnasa commented Jul 6, 2013

Apparently this is good to merge :-)
I would like to know: Was this a good approach and if so, are you planning to merge this in?

mattheath added a commit that referenced this pull request Jan 2, 2014
Add the OAuth Pecl extension.
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