diff options
author | Lukas Fleischer <archlinux@cryptocrack.de> | 2011-08-10 15:17:58 +0200 |
---|---|---|
committer | Pierre Schmitz <pierre@archlinux.de> | 2011-08-13 20:15:19 +0200 |
commit | c8ba49c6a14caff130bd64c594639bff67015b25 (patch) | |
tree | b5707b8bde64ea5ba0407d066485ac1261100e6f | |
parent | e63c5f26a2e7c29ab064f89ff1ff4a5a113a9046 (diff) | |
download | 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-x | commitpkg | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -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 |