summaryrefslogtreecommitdiff
path: root/tsp/stop.sh
diff options
context:
space:
mode:
authorAleksander Mistewicz <a.mistewicz@samsung.com>2016-08-26 14:43:02 (GMT)
committerAleksander Mistewicz <a.mistewicz@samsung.com>2016-12-06 15:37:13 (GMT)
commitd67067c8c0167e53b2adc58f3c3bc381c3bfdb50 (patch)
treeef7004b10f2977dff46f959a7db949b407f785d2 /tsp/stop.sh
parent6846bcb275df7f55c786c28854b1c7e564a7bd3e (diff)
downloadmajor-d67067c8c0167e53b2adc58f3c3bc381c3bfdb50.zip
major-d67067c8c0167e53b2adc58f3c3bc381c3bfdb50.tar.gz
major-d67067c8c0167e53b2adc58f3c3bc381c3bfdb50.tar.bz2
Fix issues reported by ShellCheck in tsprefs/changes/31/81831/12
Move check_diff() to tsp/jobs/img_test_common.sh Add newline at the end of 'modifed_paths' in tsp/scripts/crawler.py so that whole lines instead of words can be read. ShellCheck: https://github.com/koalaman/shellcheck shellcheck -e SC1090,SC2155,SC2068,SC2145,SC2164 `find tsp -name "*.sh"` output should report following problems only: tsp/common.sh: SC2148, SC2034, SC2139 tsp/jobs/trigger_downloads.sh: SC2115 tsp/jobs/common_prep_flash_conf.sh: SC2029 tsp/jobs/publish.sh: SC2046, SC2029 Change-Id: I7791769a0d653c784715cdd1a8b1446fafcba1e9 Signed-off-by: Aleksander Mistewicz <a.mistewicz@samsung.com>
Diffstat (limited to 'tsp/stop.sh')
-rwxr-xr-xtsp/stop.sh6
1 files changed, 3 insertions, 3 deletions
diff --git a/tsp/stop.sh b/tsp/stop.sh
index 77e727b..24dd63b 100755
--- a/tsp/stop.sh
+++ b/tsp/stop.sh
@@ -16,7 +16,7 @@
# Author: Aleksander Mistewicz <a.mistewicz@samsung.com>
-export TSP_DIR="$(CDPATH= cd -- "$(dirname -- "$0")" && pwd -P)"
+export TSP_DIR="$(CDPATH='' cd -- "$(dirname -- "$0")" && pwd -P)"
. "${TSP_DIR}/common.sh"
@@ -27,7 +27,7 @@ kill_running() {
echo "[$@] Kill every running job"
for i in $(eval $@ | grep "running" | grep -o "^[0-9]*")
do
- kill `eval $@ -p $i`
+ kill "$(eval $@ -p "$i")"
done
}
@@ -35,7 +35,7 @@ remove_queued() {
echo "[$@] Remove every queued job"
for i in $(eval $@ | grep "queued" | grep -o "^[0-9]*")
do
- eval $@ -r $i
+ eval $@ -r "$i"
done
}