-
Notifications
You must be signed in to change notification settings - Fork 265
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
Add parameter to specify an alternate package name to install #42
Conversation
Any thoughts on this? Its a pretty simple change. Should I implement it in a different way? |
@@ -63,16 +65,17 @@ | |||
$manage_service = true, | |||
$enable = true, | |||
$global_options = $haproxy::params::global_options, | |||
$defaults_options = $haproxy::params::defaults_options | |||
$defaults_options = $haproxy::params::defaults_options, | |||
$pkg_name = 'haproxy' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you make this the full $package_name
parameter name instead of a shortened form?
Sorry to not see this sooner! I left a comment and will try to check back soon. Feel free to ping me ( |
I updated my branch with the requested changes. |
Change parameter name per comment on pull request puppetlabs#42 puppetlabs#42 (comment)
Change parameter name per comment on pull request puppetlabs#42 puppetlabs#42 (comment)
Add parameter to specify an alternate package name to install
Change parameter name per comment on pull request puppetlabs#42 puppetlabs#42 (comment)
We have built a haproxy15 package so we can use it for ssl proxy applications. Everything else works with your module so being able to specify the name of the package to install would be very helpful. I'm sure others have similar needs.