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

apache::mod::expires class for easy including #352

Merged
merged 1 commit into from
Sep 24, 2013

Conversation

kitchen
Copy link
Contributor

@kitchen kitchen commented Sep 5, 2013

No description provided.

@blkperl
Copy link
Contributor

blkperl commented Sep 15, 2013

Why is this is PR necessary? Why not just include apache::mod { 'expires': } in your manifests?

@kitchen
Copy link
Contributor Author

kitchen commented Sep 24, 2013

So it can be included from multiple classes without conflict.

I use the role/profile pattern and each vhost is a separate class. There tends to be some overlap in the apache module dependencies :)

Insert PGP signature here.

On Sep 14, 2013, at 11:56 PM, William Van Hevelingen notifications@github.com wrote:

Why is this is PR necessary? Why not just include apache::mod { 'expires': } in your manifests?


Reply to this email directly or view it on GitHub.

hunner added a commit that referenced this pull request Sep 24, 2013
apache::mod::expires class for easy including
@hunner hunner merged commit 61e4de5 into puppetlabs:master Sep 24, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants