Skip to content

Handle Empty initdb.d directory #200

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

Merged
merged 1 commit into from
Aug 18, 2016

Conversation

apeschel
Copy link
Contributor

The docker-entrypoint.sh script does not currently handle the case where
the docker-entrypoint-initdb.d directory is empty. This commit sets the
nullglob shell option to cause the empty directory to glob to nothing.

This change will fix the following confusing message from coming up
during startup.

/usr/local/bin/docker-entrypoint.sh: ignoring /docker-entrypoint-initdb.d/*

The docker-entrypoint.sh script does not currently handle the case where
the docker-entrypoint-initdb.d directory is empty. This commit sets the
nullglob shell option to cause the empty directory to glob to nothing.

This change will fix the following confusing message from coming up
during startup.

```
/usr/local/bin/docker-entrypoint.sh: ignoring /docker-entrypoint-initdb.d/*
```
@tianon
Copy link
Member

tianon commented Aug 17, 2016

Seems sane to me -- @ltangvald?

@ltangvald
Copy link
Collaborator

LGTM. Thanks for the contribution!

@ltangvald ltangvald merged commit 79c8b1e into docker-library:master Aug 18, 2016
tianon added a commit to infosiftr/stackbrew that referenced this pull request Aug 22, 2016
- `docker`: 1.12.1
- `drupal`: 8.2.0-beta2
- `memcached`: 1.4.31
- `mysql`: handle empty `initdb` directory (docker-library/mysql#200)
- `percona`: 5.6.32, 5.5.51
- `php`: 5.6.25
- `python`: 3.6.0a4 (docker-library/python#144), `windowsservercore` (docker-library/python#142)
- `rabbitmq`: fix SSL/`ERL_ARGS` (docker-library/rabbitmq#103)
- `rocket.chat`: 0.37.1
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