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

Allow customization of the nginx package name #858

Merged
merged 2 commits into from
Aug 9, 2017

Conversation

reidab
Copy link
Contributor

@reidab reidab commented Aug 7, 2017

The nginx/development PPA includes several variants of nginx each built
with different collections of modules: nginx-core (default), nginx-light,
nginx-full, and nginx-extras.

This small change allows the nginx package installed by Trellis to be
customized via the nginx_package variable.

The nginx/development PPA includes several variants of nginx each built
with different collections of modules: nginx-core (default), nginx-light,
nginx-full, and nginx-extras.

This small change allows the nginx package installed by Trellis to be
customized via the `nginx_package` variable.
@tangrufus
Copy link
Collaborator

Should we make apt repo customisable as well?

- name: Add Nginx PPA
  apt_repository:
    repo: "ppa:nginx/development" <------ this one
    update_cache: yes

@reidab
Copy link
Contributor Author

reidab commented Aug 7, 2017

@tangrufus I think so. I almost did this initially — I'll update shortly.

@swalkinshaw swalkinshaw merged commit c8b6ceb into roots:master Aug 9, 2017
@swalkinshaw
Copy link
Member

Thanks 🚀

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.

3 participants