[sr-dev] git:5.2:d55905c1: utils: Change sh(1) "test ==" usage to "="

Module: kamailio Branch: 5.2 Commit: d55905c1877b13f5d8367e8d4fd16cd0f01dffee URL: https://github.com/kamailio/kamailio/commit/d55905c1877b13f5d8367e8d4fd16cd… Author: Greg Troxel &lt;gdt(a)lexort.com&gt; Committer: Henning Westerholt &lt;hw(a)skalatan.de&gt; Date: 2019-11-29T18:42:04+01:00 utils: Change sh(1) "test ==" usage to "=" The POSIX sh(1) specification says that strings are compared with test(1) (also "[") with "=". Bash accepts "==" and this leads to non-portable code. This commit simply changes "==" within test/[ to "=". (cherry picked from commit e8b8251f6c7382f8e373c2b309422973a3bf730d) --- Modified: utils/kamctl/kamdbctl --- Diff: https://github.com/kamailio/kamailio/commit/d55905c1877b13f5d8367e8d4fd16cd… Patch: https://github.com/kamailio/kamailio/commit/d55905c1877b13f5d8367e8d4fd16cd… --- diff --git a/utils/kamctl/kamdbctl b/utils/kamctl/kamdbctl index ae44c5df45..7a7cfaa7f7 100755 --- a/utils/kamctl/kamdbctl +++ b/utils/kamctl/kamdbctl @@ -153,7 +153,7 @@ kamailio_dump() # pars: <database name> merr "kamailio_dump function takes two param" exit 1 fi - if [ "$USED_DBENGINE" == "oracle" ]; then + if [ "$USED_DBENGINE" = "oracle" ]; then oracle_dump $1 $2 elif [ "$PW" = "" ] ; then $DUMP_CMD $1 > $2 @@ -174,7 +174,7 @@ kamailio_restore() #pars: <database name> <filename> merr "kamailio_restore function takes two params" exit 1 fi - if [ "$USED_DBENGINE" == "oracle" ]; then + if [ "$USED_DBENGINE" = "oracle" ]; then oracle_restore $1 $2 else sql_query $1 < $2 @@ -290,7 +290,7 @@ kamailio_pframework() #pars: <action> case $1 in copy) # copy database to some other name - if [ "$USED_DBENGINE" == "berkeley" -o "$USED_DBENGINE" == "dbtext" ] ; then + if [ "$USED_DBENGINE" = "berkeley" -o "$USED_DBENGINE" = "dbtext" ] ; then merr "$USED_DBENGINE don't support this operation" exit 1 fi @@ -323,7 +323,7 @@ case $1 in exit $ret ;; backup) - if [ "$USED_DBENGINE" == "berkeley" -o "$USED_DBENGINE" == "dbtext" ] ; then + if [ "$USED_DBENGINE" = "berkeley" -o "$USED_DBENGINE" = "dbtext" ] ; then merr "$USED_DBENGINE don't support this operation" exit 1 fi @@ -337,7 +337,7 @@ case $1 in exit $? ;; restore) - if [ "$USED_DBENGINE" == "berkeley" -o "$USED_DBENGINE" == "dbtext" ] ; then + if [ "$USED_DBENGINE" = "berkeley" -o "$USED_DBENGINE" = "dbtext" ] ; then merr "$USED_DBENGINE don't support this operation" exit 1 fi