Skip to content

Commit 2966bfb

Browse files
committed
Add consistent variable for "--verbose --help" arguments
Given these are duplicated (or should be -- as seen in the diff we already messed this up!) it makes sense to make them share a common variable.
1 parent c273193 commit 2966bfb

File tree

4 files changed

+32
-8
lines changed

4 files changed

+32
-8
lines changed

5.6/docker-entrypoint.sh

Lines changed: 8 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

5.7/docker-entrypoint.sh

Lines changed: 8 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

8.0/docker-entrypoint.sh

Lines changed: 8 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

template/docker-entrypoint.sh

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,14 @@ docker_process_init_files() {
7878
done
7979
}
8080

81+
# arguments necessary to run "mysqld --verbose --help" successfully (used for testing configuration validity and for extracting default/configured values)
82+
_verboseHelpArgs=(
83+
--verbose --help
84+
--log-bin-index="$(mktemp -u)" # https://github.com/docker-library/mysql/issues/136
85+
)
86+
8187
mysql_check_config() {
82-
local toRun=( "$@" --verbose --help ) errors
88+
local toRun=( "$@" "${_verboseHelpArgs[@]}" ) errors
8389
if ! errors="$("${toRun[@]}" 2>&1 >/dev/null)"; then
8490
mysql_error $'mysqld failed while attempting to check config\n\tcommand was: '"${toRun[*]}"$'\n\t'"$errors"
8591
fi
@@ -90,7 +96,7 @@ mysql_check_config() {
9096
# latter only show values present in config files, and not server defaults
9197
mysql_get_config() {
9298
local conf="$1"; shift
93-
"$@" --verbose --help --log-bin-index="$(mktemp -u)" 2>/dev/null \
99+
"$@" "${_verboseHelpArgs[@]}" 2>/dev/null \
94100
| awk -v conf="$conf" '$1 == conf && /^[^ \t]/ { sub(/^[^ \t]+[ \t]+/, ""); print; exit }'
95101
# match "datadir /some/path with/spaces in/it here" but not "--xyz=abc\n datadir (xyz)"
96102
}

0 commit comments

Comments
 (0)