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

Flags parameter supported only on OpenBSD #569

Merged
merged 2 commits into from
Mar 26, 2015
Merged

Flags parameter supported only on OpenBSD #569

merged 2 commits into from
Mar 26, 2015

Conversation

Zophar78
Copy link
Contributor

Flags parameter is only supported on OpenBSD and breaks on some other systems like RH (on puppet 3.4.x)
See the discussion here: 2960c10cd

Zophar78 referenced this pull request Feb 12, 2015
Give example for passenger in README.md and add OpenBSD to the
supported OSs in metadata.json.
ensure => $service_ensure_real,
name => $service_name,
enable => $service_enable,
flags => '',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't tested, but this line should be:

flags => $service_flags,

otherwise looks exactly as what I would have done.

@Zophar78
Copy link
Contributor Author

Will do a branch next time (i did that a bit too fast)

@jrgarcia
Copy link

👍 I'd love to see this get pulled in. This module no longer works on our CentOS systems.

jfryman added a commit that referenced this pull request Mar 26, 2015
Flags parameter supported only on OpenBSD
@jfryman jfryman merged commit ef98b63 into voxpupuli:master Mar 26, 2015
@jrgarcia
Copy link

Thanks so much!

@3flex 3flex mentioned this pull request Apr 1, 2015
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this pull request Oct 23, 2017
Flags parameter supported only on OpenBSD
Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this pull request Mar 7, 2019
Flags parameter supported only on OpenBSD
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.

4 participants