summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <archlinux@cryptocrack.de>2011-08-10 15:17:58 +0200
committerPierre Schmitz <pierre@archlinux.de>2011-08-13 20:15:19 +0200
commitc8ba49c6a14caff130bd64c594639bff67015b25 (patch)
treeb5707b8bde64ea5ba0407d066485ac1261100e6f
parente63c5f26a2e7c29ab064f89ff1ff4a5a113a9046 (diff)
downloaddevtools-c8ba49c6a14caff130bd64c594639bff67015b25.tar.gz
devtools-c8ba49c6a14caff130bd64c594639bff67015b25.tar.xz
commitpkg: Proper quoting in version control checks
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de> Signed-off-by: Pierre Schmitz <pierre@archlinux.de>
-rwxr-xr-xcommitpkg8
1 files changed, 4 insertions, 4 deletions
diff --git a/commitpkg b/commitpkg
index 9856df1..999c35f 100755
--- a/commitpkg
+++ b/commitpkg
@@ -78,9 +78,9 @@ case "$repo" in
esac
# check if all local source files are under version control
-for s in ${source[@]}; do
- echo $s | grep -Fvq '://' && \
- svn status $s | grep -q '^\?' && \
+for s in "${source[@]}"; do
+ echo "$s" | grep -Fvq '://' && \
+ svn status "$s" | grep -q '^\?' && \
abort "$s is not under version control"
done
@@ -90,7 +90,7 @@ for i in 'changelog' 'install'; do
for file in $filelist; do
# evaluate any bash variables used
eval file=${file}
- if svn status ${file} | grep -q '^\?'; then
+ if svn status "${file}" | grep -q '^\?'; then
abort "${file} is not under version control"
fi
done