Skip to content

Commit

Permalink
make-binaries: Allow for empty INSTALL_DIR_FOR_OTP
Browse files Browse the repository at this point in the history
There's probably no actual use case for setting INSTALL_DIR_FOR_OTP to
an empty value, this change is just for consistency with ejabberd's
tools/make-binaries script.
  • Loading branch information
weiss committed Jul 23, 2023
1 parent c06b9af commit 51eb2ab
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/make-binaries
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ add_otp_path()
if [ "$mode" = 'native' ]
then
native_otp_bin="$prefix/bin"
elif [ -n "${INSTALL_DIR_FOR_OTP:+x}" ]
elif [ -n "${INSTALL_DIR_FOR_OTP+x}" ]
then
# Let GitHub runners build for non-native systems:
# https://github.com/erlef/setup-beam#environment-variables
Expand Down Expand Up @@ -563,7 +563,7 @@ else
targets="$*"
fi

if [ "$build_native" = 'false' ] && [ -z "${INSTALL_DIR_FOR_OTP:+x}" ]
if [ "$build_native" = 'false' ] && [ -z "${INSTALL_DIR_FOR_OTP+x}" ]
then
error 'Native Erlang/OTP installation not found.'
error "Set INSTALL_DIR_FOR_OTP or add the '$platform' target."
Expand Down

0 comments on commit 51eb2ab

Please sign in to comment.