termux-api: Cleanups and fix whitespace handling

Fixes https://github.com/termux/termux-api/issues/20

diff --git a/packages/termux-api/build.sh b/packages/termux-api/build.sh
index 8746517997c08a492113f28864fbdfbe9b2165bd..9a51a1b051ac5b11741c30a3ff9a4c68194769e8 100644 (file)
--- a/packages/termux-api/build.sh
+++ b/packages/termux-api/build.sh
@@ -1,6 +1,6 @@
 TERMUX_PKG_HOMEPAGE=http://termux.com/add-ons/api/
 TERMUX_PKG_DESCRIPTION="Termux API commands"
 TERMUX_PKG_HOMEPAGE=http://termux.com/add-ons/api/
 TERMUX_PKG_DESCRIPTION="Termux API commands"

-TERMUX_PKG_VERSION=0.16
+TERMUX_PKG_VERSION=0.17

 
 termux_step_make_install () {
         mkdir -p $TERMUX_PREFIX/bin
 
 termux_step_make_install () {
         mkdir -p $TERMUX_PREFIX/bin

diff --git a/packages/termux-api/termux-dialog b/packages/termux-api/termux-dialog
index d9684533dbbc98a526d4d01c8fad9bc56f7c02fc..c513f9442959ff6bfe587ade62a0559edadf6d29 100755 (executable)
--- a/packages/termux-api/termux-dialog
+++ b/packages/termux-api/termux-dialog
@@ -6,10 +6,10 @@ show_usage () {
        echo "Usage: $SCRIPTNAME [OPTIONS]"
        echo "Show a text entry dialog."
        echo ""
        echo "Usage: $SCRIPTNAME [OPTIONS]"
        echo "Show a text entry dialog."
        echo ""

-       echo "  -i <hint>   The input hint to show when the input is empty"
-       echo "  -m          Use a textarea with multiple lines instead of a single"
-       echo "  -p          Enter the input as a password"
-       echo "  -t <title>  The title to show for the input prompt"
+       echo "  -i <hint>   the input hint to show when the input is empty"
+       echo "  -m          use a textarea with multiple lines instead of a single"
+       echo "  -p          enter the input as a password"
+       echo "  -t <title>  the title to show for the input prompt"

        exit 0
 }
 
        exit 0
 }
 


@@ -37,4 +37,8 @@ shift $(($OPTIND-1))
 
 if [ $# != 0 ]; then echo "$SCRIPTNAME: too many arguments"; exit 1; fi
 
 
 if [ $# != 0 ]; then echo "$SCRIPTNAME: too many arguments"; exit 1; fi
 

-@TERMUX_API@ Dialog $ARG_I "$OPT_I" $ARG_M $ARG_P $ARG_T "$OPT_T"
+set -- $ARG_M $ARG_P
+if [ -n "$ARG_I" ]; then set -- "$@" $ARG_I "$OPT_I"; fi
+if [ -n "$ARG_T" ]; then set -- "$@" $ARG_T "$OPT_T"; fi
+
+@TERMUX_API@ Dialog "$@"

diff --git a/packages/termux-api/termux-download b/packages/termux-api/termux-download
index c7b8fea9839e1173ba63d433317e4c4a08ba1399..f8e3f976dc696777c8339754fa3c777f77245ff3 100755 (executable)
--- a/packages/termux-api/termux-download
+++ b/packages/termux-api/termux-download
@@ -1,30 +1,40 @@
 #!/bin/sh
 #!/bin/sh

-

 set -e -u
 
 set -e -u
 

-SCRIPTNAME=$0
+SCRIPTNAME=termux-download

 show_usage () {
 show_usage () {

-       echo "usage: termux-download <uri-to-download>"
-       echo "       Download a resource using the system download manager."
-       echo "       -d, --description      Description for the download request notification"
-       echo "       -t, --title            Title for the download request notification"
+       echo "Usage: $SCRIPTNAME [OPTIONS] <url-to-download>"
+       echo "Download a resource using the system download manager."
+       echo ""
+       echo "  -d <description>  description for the download request notification"
+       echo "  -t <title>        title for the download request notification"
+       exit 0

 }
 
 }
 

-PARAMS=""
-O=`busybox getopt -q -l description: -l title: -l help -- d:t:h "$@"`
-if [ $? != 0 ] ; then show_usage; exit 1 ; fi
-eval set -- "$O"
-while true; do
-case "$1" in
-       -h|--help) show_usage; exit 0;;
-       -d|--description) PARAMS="$PARAMS --es description '$2'"; shift 2;;
-       -t|--title) PARAMS="$PARAMS --es title '$2'"; shift 2;;
-       --)     shift; break;;
-       *)      echo Error; exit 1;;
-esac
+ARG_D=""
+OPT_D=""
+ARG_T=""
+OPT_T=""
+
+while getopts :hd:t: option
+do
+    case "$option" in
+       h) show_usage;;
+       d) ARG_D="--es description"; OPT_D="$OPTARG";;
+       t) ARG_T="--es title"; OPT_T="$OPTARG";;
+       ?) echo "$SCRIPTNAME: illegal option -$OPTARG"; exit 1;
+    esac

 done
 done

+shift $(($OPTIND-1))
+
+if [ $# -lt 1 ]; then echo "$SCRIPTNAME: no url specified"; exit 1; fi
+if [ $# -gt 1 ]; then echo "$SCRIPTNAME: too many arguments"; exit 1; fi
+
+URL_TO_DOWNLOAD="$1"

 
 

-# Too many arguments:
-if [ $# != 1 ]; then show_usage; exit 1; fi
+set --
+if [ -n "$ARG_D" ]; then set -- "$@" $ARG_D "$OPT_D"; fi
+if [ -n "$ARG_T" ]; then set -- "$@" $ARG_T "$OPT_T"; fi
+set -- "$@" $URL_TO_DOWNLOAD

 
 

-eval @TERMUX_API@ Download $PARAMS $1
+@TERMUX_API@ Download "$@"

diff --git a/packages/termux-api/termux-notification b/packages/termux-api/termux-notification
index 96a0e2a53e0902fa9f0879fbfea032b6777bc538..34a5aab3434269b5e9e5138473ff17aab5e9c12d 100755 (executable)
--- a/packages/termux-api/termux-notification
+++ b/packages/termux-api/termux-notification
@@ -3,7 +3,7 @@ set -e -u
 
 SCRIPTNAME=termux-notification
 show_usage () {
 
 SCRIPTNAME=termux-notification
 show_usage () {

-       echo "usage: termux-notification [OPTIONS]"
+       echo "Usage: termux-notification [OPTIONS]"

        echo "Display a system notification."
        echo ""
        echo "  -c <content> notification content to show"
        echo "Display a system notification."
        echo ""
        echo "  -c <content> notification content to show"


@@ -43,4 +43,10 @@ if [ $CONTENT_OR_TITLE_SET = "no" ]; then
        exit 1
 fi
 
        exit 1
 fi
 

-@TERMUX_API@ Notification $ARG_C "$OPT_C" $ARG_I "$OPT_I" $ARG_T "$OPT_T" $ARG_U "$OPT_U"
+set --
+if [ -n "$ARG_C" ]; then set -- "$@" $ARG_C "$OPT_C"; fi
+if [ -n "$ARG_I" ]; then set -- "$@" $ARG_I "$OPT_I"; fi
+if [ -n "$ARG_T" ]; then set -- "$@" $ARG_T "$OPT_T"; fi
+if [ -n "$ARG_U" ]; then set -- "$@" $ARG_U "$OPT_U"; fi
+
+@TERMUX_API@ Notification "$@"

diff --git a/packages/termux-api/termux-sms-send b/packages/termux-api/termux-sms-send
index 43664a5c3043d3beae5fb894e13ee3c94040c75a..7802b07b4ea356038793d66ddcb563fb25e73836 100755 (executable)
--- a/packages/termux-api/termux-sms-send
+++ b/packages/termux-api/termux-sms-send
@@ -6,7 +6,7 @@ show_usage () {
        echo "Usage: $SCRIPTNAME [-t <text>] <recipient-number>"
        echo "Send a SMS."
        echo ""
        echo "Usage: $SCRIPTNAME [-t <text>] <recipient-number>"
        echo "Send a SMS."
        echo ""

-       echo "  -t <text>  The text to send (optional - else from stdin)"
+       echo "  -t <text>  text to send (optional - else from stdin)"

        echo ""
        echo "If no text is specified with the -t option the text to send is read from stdin."
        exit 0
        echo ""
        echo "If no text is specified with the -t option the text to send is read from stdin."
        exit 0