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

Add Archlinux support #316

Merged
merged 2 commits into from May 16, 2014
Merged

Add Archlinux support #316

merged 2 commits into from May 16, 2014

Conversation

ghost
Copy link

@ghost ghost commented May 15, 2014

This commit adds Archlinux support. This needs changes in nginx.conf.erb
template and manifests to support pid value set to false (this skips it from
nginx.conf). This is due to -pid /run/nginx.pid being hardcoded in Archlinux
nginx service file. If you specify pid additionally in nginx.conf then nginx
service fails to start.

I have also fixed some typos and removed whitespaces in second commit
from various files. Feel free to cherry-pick only the first one if you feel like it.

Edvinas Klovas added 2 commits May 15, 2014 23:55
This commit adds Archlinux support. This needs changes in nginx.conf.erb
template and manifests to support pid value set to `false` (this skips it from
nginx.conf). This is due to `-pid /run/nginx.pid` being hardcoded to Archlinux
template. If you specify pid additionally in `nginx.conf` then nginx fails to
start.
@jfryman
Copy link
Contributor

jfryman commented May 16, 2014

Neat! Thanks for the code.

jfryman pushed a commit that referenced this pull request May 16, 2014
@jfryman jfryman merged commit 5cb8ea8 into voxpupuli:master May 16, 2014
@ghost ghost deleted the archlinux branch May 16, 2014 21:20
@ghost ghost restored the archlinux branch May 16, 2014 21:31
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.

1 participant