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

Use package pecl-memcache on RHEL #56

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

Conversation

christopher-hopper
Copy link

When installing the memcache php module on RHEL, use the package 'php-pecl-memcache' in a similar way to the existing apc php module.

NOTE if more of these exceptions to package names are encountered, it might be worth defining all module package names in a hash or array, to simplify the branching and php::param variables.

Use similar parameter for memcache as we use for apc when installing on
RHEL because they're both coming as pecl packages.
@thias
Copy link
Owner

thias commented Mar 6, 2015

I'm using both the php-pecl-memcache and php-pecl-memcached modules on RHEL, and have never needed an exception. What's wrong with just the following?

php::module { 'pecl-memcache': }

@thias thias self-assigned this Mar 6, 2015
@christopher-hopper
Copy link
Author

I guess that the problem is that we want this defined type to not require OS or Distro specific patterns in order to use. My puppet manifests will be used on Debian and RHEL builds, so I'd like to be able to use:

php::module { 'memcache': }

In the same way I can use:

php::module { 'apc': }

and have it just work on Debian or RHEL. Does that make sense?

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