diff options
author | Phil Pennock <pdp@exim.org> | 2013-11-20 17:16:02 -0800 |
---|---|---|
committer | Phil Pennock <pdp@exim.org> | 2013-11-20 17:16:02 -0800 |
commit | 5c7706b0c39a7cbb031f1d86594b41e0c96008fa (patch) | |
tree | fadd60457b3152dbcac7d50f7c8cb79209b28be2 /src | |
parent | a4ca1b01d4376f9b9ca7e06acf4eb7d164a38d2b (diff) |
build: try to get dash/bash for sanity
The "local" builtin is not part of POSIX. We want it. Try harder to
get a vaguely sane shell, rather than just a POSIX shell.
Also, safeguard to error out more gracefully if invoked from outside the
build process.
Diffstat (limited to 'src')
-rwxr-xr-x | src/scripts/lookups-Makefile | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/src/scripts/lookups-Makefile b/src/scripts/lookups-Makefile index d2caf70c8..61493c632 100755 --- a/src/scripts/lookups-Makefile +++ b/src/scripts/lookups-Makefile @@ -24,6 +24,22 @@ then _XPG=1 export _XPG + # We need the _right_ tr, so must do that first; but if a shell which + # we're more confident is sane is available, let's try that. Mostly, + # the problem is that "local" is not actually in "the" standard, it's + # just in every not-insane shell. Though arguably, there are no shells + # with POSIX-ish syntax which qualify as "not insane". + for b in /bin/dash /bin/bash /usr/local/bin/bash + do + if [ -x "$b" ] + then + SHELL="$b" + break + fi + done + # if we get a report of a system with zsh but not bash, we can add that + # to the list, but be sure to enable sh_word_split in that case. + exec "$SHELL" "$0" "$@" fi @@ -41,6 +57,16 @@ tab=' ' LC_ALL=C export LC_ALL +if [ -f "$defs_source" ] +then + : + # we are happy +else + echo >&2 "$0: ERROR: MISSING FILE '${defs_source}'" + echo >&2 "$0: SHOULD HAVE BEEN CALLED FROM scripts/Configure-Makefile" + exit 1 +fi + # nb: do not permit leading whitespace for this, as CFLAGS_DYNAMIC is exported # to the lookups subdir via a line with leading whitespace which otherwise # matches |