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

Custom optional custom file. #72

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

luison
Copy link

@luison luison commented Jan 12, 2019

Hi. I can't say I love bash but I've modified the script to support the optional load of a file nginx-custom.options that can include any of both of your NGINX_MODULES and NGINX_OPTIONS variables.

This allows to personalise the standard compile modules (was my case) without loosing the possibility to update your script or having to modify it again as I've add it to the git ignore list. Left a "sample" one.

Feel free to include it if you find it of interest.

I've had to redo it as I've noticed you've, just today, modified considerably the code. Is done in any case although I'll admit I liked better your previous "echo" style and log interface.

By the way I only found this today which is not updated but might be of interest to you too: https://github.com/naologic/nginx-builder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants