Skip to content
This repository was archived by the owner on Aug 19, 2019. It is now read-only.

The entrypoint script uses bash expressions, so should be run with /bin/bash.#161

Merged
igorpeshansky merged 2 commits intomasterfrom
igorp-fix-entrypoint
Jul 13, 2018
Merged

The entrypoint script uses bash expressions, so should be run with /bin/bash.#161
igorpeshansky merged 2 commits intomasterfrom
igorp-fix-entrypoint

Conversation

@igorpeshansky
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@jkohen jkohen left a comment

Choose a reason for hiding this comment

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

Assuming /bin/bash is always available in the Dockerfile-xenial base image, this makes sense to me.

It'd be good to document what the bashisms are, if you've already identified them. Otherwise no worries.

@igorpeshansky
Copy link
Copy Markdown
Contributor Author

Addressed comments. PTAL.

Copy link
Copy Markdown
Contributor

@bmoyles0117 bmoyles0117 left a comment

Choose a reason for hiding this comment

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

LGTM

@igorpeshansky igorpeshansky removed the request for review from rbuskens July 13, 2018 16:31
@igorpeshansky igorpeshansky merged commit 2774c57 into master Jul 13, 2018
@igorpeshansky igorpeshansky deleted the igorp-fix-entrypoint branch July 13, 2018 16:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants