Skip to content

Commit ff49bf3

Browse files
committed
fix numerical check
1 parent 22aa469 commit ff49bf3

File tree

1 file changed

+15
-11
lines changed

1 file changed

+15
-11
lines changed

bin/run-clara

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#!/usr/bin/env bash
1+
#!/usr/bin/env zsh
22

33
ulimit -u 49152 >& /dev/null
44
export JAVA_OPTS="${JAVA_OPTS} -XX:+IgnoreUnrecognizedVMOptions"
@@ -30,22 +30,25 @@ function get_host_ip() {
3030
then
3131
while IFS=$': \t' read -r -a line
3232
do
33-
[ -z "${line%inet}" ] &&
34-
ip=${line[${#line[1]}>4?1:2]} &&
35-
[ "${ip#127.0.0.1}" ] && echo $ip && return 0
33+
if [ -z "${line%inet}" ]
34+
then
35+
ip=${line[${#line[1]}>4?1:2]}
36+
[ "${ip#127.0.0.1}" ]
37+
echo $ip && return 0
38+
fi
3639
done< <(LANG=C ifconfig)
3740
fi
3841
return 1
3942
}
4043
function get_dpe_port() {
4144
local ports
4245
ports=$(seq 7000 20 8000)
43-
if command -v shuf >/dev/null 2>&1; then
44-
ports=$(echo "$ports" | shuf)
45-
fi
46-
for port in $ports; do
46+
command -v shuf >/dev/null 2>&1 && ports=$(echo "$ports" | shuf)
47+
for port in $ports
48+
do
4749
local ctrl_port=$((port + 2))
48-
if ! eval "exec 6<>/dev/tcp/127.0.0.1/$ctrl_port" 2> /dev/null; then
50+
if ! eval "exec 6<>/dev/tcp/127.0.0.1/$ctrl_port" 2> /dev/null
51+
then
4952
echo $port
5053
return 0
5154
fi
@@ -65,11 +68,12 @@ do
6568
i) input=$OPTARG ;;
6669
o) output=$OPTARG ;;
6770
c) CLARA_HOME=$OPTARG ;;
68-
t) threads=$OPTARG && [[ $threads =~ '^[0-9]+$' ]] || error "-t must be an integer, threads" ;;
69-
n) nevents="-e $OPTARG" && [[ $nevents =~ '^[0-9]+$' ]] || error "-n must be an integer, events" ;;
71+
t) threads=$OPTARG && echo $threads | grep -q -E '^[0-9]+$' || error "-t must be an integer, threads" ;;
72+
n) nevents="-e $OPTARG" && echo $nevents | grep -q -E '^-e [0-9]+$' || error "-n must be an integer, events" ;;
7073
h) echo -e $usage && echo -e $info && exit 0 ;;
7174
esac
7275
done
76+
7377
shift $((OPTIND-1))
7478
[ $# -gt 1 ] && error "Extra arguments: ${@:2} (options must come before positionals)"
7579
[ $# -lt 1 ] && error "YAML file argument is required"

0 commit comments

Comments
 (0)