[Asterisk-code-review] pjproject: Remove bashism from configure.m4 script (asterisk[13])

[Guido Falsi]

Guido Falsi has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/14336 )


Change subject: pjproject: Remove bashism from configure.m4 script
......................................................................

pjproject: Remove bashism from configure.m4 script

The configure.m4 script for pjproject contains some += syntax, which
is specific to bash, replacing it with string substitutions makes
the script compatible with traditional Bourne shells.

ASTERISK-28866 #close
Reported-by: Christoph Moench-Tegeder <cmt at FreeBSD.org>

Change-Id: I382a78160e028044598b7da83ec7e1ff42b91c05
---
M third-party/pjproject/configure.m4
1 file changed, 5 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/36/14336/1

diff --git a/third-party/pjproject/configure.m4 b/third-party/pjproject/configure.m4
index 7cbb307..485e4f1 100644
--- a/third-party/pjproject/configure.m4
+++ b/third-party/pjproject/configure.m4
@@ -51,25 +51,25 @@
 		AC_ARG_VAR([PJPROJECT_CONFIGURE_OPTS],[Additional configure options to pass to bundled pjproject])
 		this_host=$(./config.sub $(./config.guess))
 		if test "$build" != "$this_host" ; then
-			PJPROJECT_CONFIGURE_OPTS+=" --build=$build_alias"
+			PJPROJECT_CONFIGURE_OPTS="${PJPROJECT_CONFIGURE_OPTS} --build=$build_alias"
 		fi
 		if test "$host" != "$this_host" ; then
-			PJPROJECT_CONFIGURE_OPTS+=" --host=$host_alias"
+			PJPROJECT_CONFIGURE_OPTS="${PJPROJECT_CONFIGURE_OPTS} --host=$host_alias"
 		fi
 		# This was a copy of the autoconf generated code from the root ./configure.
 		# Hopefully, when you read this, the code is still the same.
 		if test "${with_ssl+set}" = set; then :
 			case $with_ssl in
 			n|no)
-			PJPROJECT_CONFIGURE_OPTS+=" --disable-ssl"
+			PJPROJECT_CONFIGURE_OPTS="${PJPROJECT_CONFIGURE_OPTS} --disable-ssl"
 			;;
 			y|ye|yes)
 			# Not to mention SSL is the default in PJProject and means "autodetect".
 			# In Asterisk, "./configure --with-ssl" means "must be present".
-			PJPROJECT_CONFIGURE_OPTS+=""
+			PJPROJECT_CONFIGURE_OPTS="${PJPROJECT_CONFIGURE_OPTS}"
 			;;
 			*)
-			PJPROJECT_CONFIGURE_OPTS+=" --with-ssl=${with_ssl}"
+			PJPROJECT_CONFIGURE_OPTS="${PJPROJECT_CONFIGURE_OPTS} --with-ssl=${with_ssl}"
 			;;
 			esac
 		fi

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/14336
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I382a78160e028044598b7da83ec7e1ff42b91c05
Gerrit-Change-Number: 14336
Gerrit-PatchSet: 1
Gerrit-Owner: Guido Falsi <madpilot at freebsd.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200503/334b57f6/attachment.html>