-
Notifications
You must be signed in to change notification settings - Fork 864
using sh to make alpine images work #6
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
Conversation
Alpine does not come with bash installed, so "command" should use sh to be compatible with the alpine version of nginx images.
Hello @MrApe ,
I do not have time to look at this but I will analyze your request as soon as possible.
Thanks for your feedback.
I just looked at your sources and maybe you have encountered errors about the shell to use? I did not have any worries about that.
It is also the bash which is used in the documentation of Nginx.
Can you give me more information about this choice please?
Hi @MrApe,
I come back after a while, but as there was an error in the test, I have not integrated your pull request.
You're right about the fact that it generates an error if I go through the bash and it was the case because I did not use the image nginx: alpine
The changes will be made in the next release.
Thanks for your feedback.
Vincent.
Alpine does not come with bash installed, so "command" should use sh to be compatible with the alpine version of nginx images.