summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAaron Lindsay <aaron@aclindsay.com>2013-09-05 20:46:29 +0200
committerAllan McRae <allan@archlinux.org>2013-09-18 06:28:03 +0200
commit3f99cfba1b6ef1c5665488b651a3d6409730fa98 (patch)
tree453c75070ae7b3b2a4b095bdc50c340819084aa6
parent93888089643b017390538d1f6f3a2d2aa9918851 (diff)
downloadpacman-3f99cfba1b6ef1c5665488b651a3d6409730fa98.tar.gz
pacman-3f99cfba1b6ef1c5665488b651a3d6409730fa98.tar.xz
contrib: Unify quoting in error messages
This fixes error messages to display the contents of variables rather than the variable name by replacing backticks with single quotes (m4 eats backticks). It also removes $"" localization from error messages to eliminate security holes. For instance, `rankmirrors nonexistent_file' will now display: 'nonexistent_file' does not exist. rather than: $1 does not exist. Signed-off-by: Aaron Lindsay <aaron@aclindsay.com> Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r--contrib/bacman.sh.in2
-rw-r--r--contrib/rankmirrors.sh.in10
-rw-r--r--contrib/updpkgsums.sh.in2
3 files changed, 7 insertions, 7 deletions
diff --git a/contrib/bacman.sh.in b/contrib/bacman.sh.in
index 5435e405..33e3920a 100644
--- a/contrib/bacman.sh.in
+++ b/contrib/bacman.sh.in
@@ -124,7 +124,7 @@ fi
if [[ ! -d $pkg_dir ]]; then
error "package %s is found in pacman database," "${pkg_name}"
- plain " but \`%s' is not a directory" "${pkg_dir}"
+ plain " but '%s' is not a directory" "${pkg_dir}"
exit 1
fi
diff --git a/contrib/rankmirrors.sh.in b/contrib/rankmirrors.sh.in
index 875a1439..5555edf9 100644
--- a/contrib/rankmirrors.sh.in
+++ b/contrib/rankmirrors.sh.in
@@ -127,7 +127,7 @@ while [[ $1 ]]; do
verbose) VERBOSE=1 ; shift ;;
url) CHECKURL=1; [[ $2 ]] || err "Must specify url."; URL="$2"; shift 2;;
repo) [[ $2 ]] || err "Must specify repo name."; TARGETREPO="$2"; shift 2;;
- *) err "\`$1' is an invalid argument."
+ *) err "'$1' is an invalid argument."
esac
elif [[ ${1:0:1} = - ]]; then
@@ -146,7 +146,7 @@ while [[ $1 ]]; do
u) CHECKURL=1; [[ $2 ]] || err "Must specify url."; URL="$2"; snum=2;;
r) [[ $2 ]] || err "Must specify repo name."; TARGETREPO="$2"; snum=2;;
n) [[ $2 ]] || err "Must specify number." ; NUM="$2" ; snum=2;;
- *) err "\`-$1' is an invald argument." ;;
+ *) err "'$1' is an invalid argument." ;;
esac
done
shift $snum
@@ -157,7 +157,7 @@ while [[ $1 ]]; do
[[ $linearray ]] || err "File is empty."
shift
else
- err "\`$1' does not exist."
+ err "'$1' does not exist."
fi
done
@@ -169,7 +169,7 @@ done
# Single url handling
if [[ $CHECKURL ]]; then
url="$(getfetchurl "$URL")"
- [[ $url = fail ]] && err "url \`$URL' is malformed."
+ [[ $url = fail ]] && err "url '$URL' is malformed."
[[ $VERBOSE ]] && echo "Testing $url..."
time=$(gettime "$url")
echo "$URL : $time"
@@ -193,7 +193,7 @@ for line in "${linearray[@]}"; do
server="${line#*= }"
server="${server%%#*}"
url="$(getfetchurl "$server")"
- [[ $url = fail ]] && err "url \`$URL' is malformed."
+ [[ $url = fail ]] && err "url '$URL' is malformed."
time=$(gettime "$url")
timesarray+=("$time $server")
diff --git a/contrib/updpkgsums.sh.in b/contrib/updpkgsums.sh.in
index ffea96b6..4d35357f 100644
--- a/contrib/updpkgsums.sh.in
+++ b/contrib/updpkgsums.sh.in
@@ -67,7 +67,7 @@ fi
# Check $PWD/ for permission to unlink the $buildfile and write a new one
if [[ ! -w . ]]; then
- printf $'==> ERROR: No write permission in `%s\'\n' "$PWD"
+ printf "==> ERROR: No write permission in '%s'\n" "$PWD"
exit 1
fi