makepkg: use printf rather than echo to output variable
Also make sure the strings passed to %s in printf are always quoted. Signed-off-by: Allan McRae <allan@archlinux.org> Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
parent
d11192c73a
commit
1917c845f7
|
@ -211,7 +211,7 @@ get_filepath() {
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
echo "$file"
|
printf "%s\n" "$file"
|
||||||
}
|
}
|
||||||
|
|
||||||
# Print 'source not found' error message and exit makepkg
|
# Print 'source not found' error message and exit makepkg
|
||||||
|
@ -226,13 +226,13 @@ get_filename() {
|
||||||
# if a filename is specified, use it
|
# if a filename is specified, use it
|
||||||
local filename="${1%%::*}"
|
local filename="${1%%::*}"
|
||||||
# if it is just an URL, we only keep the last component
|
# if it is just an URL, we only keep the last component
|
||||||
echo "${filename##*/}"
|
printf "%s\n" "${filename##*/}"
|
||||||
}
|
}
|
||||||
|
|
||||||
# extract the URL from a source entry
|
# extract the URL from a source entry
|
||||||
get_url() {
|
get_url() {
|
||||||
# strip an eventual filename
|
# strip an eventual filename
|
||||||
echo "${1#*::}"
|
printf "%s\n" "${1#*::}"
|
||||||
}
|
}
|
||||||
|
|
||||||
##
|
##
|
||||||
|
@ -242,9 +242,9 @@ get_url() {
|
||||||
get_full_version() {
|
get_full_version() {
|
||||||
if [[ -z $1 ]]; then
|
if [[ -z $1 ]]; then
|
||||||
if [[ $epoch ]] && (( ! $epoch )); then
|
if [[ $epoch ]] && (( ! $epoch )); then
|
||||||
echo $pkgver-$pkgrel
|
printf "%s\n" "$pkgver-$pkgrel"
|
||||||
else
|
else
|
||||||
echo $epoch:$pkgver-$pkgrel
|
printf "%s\n" "$epoch:$pkgver-$pkgrel"
|
||||||
fi
|
fi
|
||||||
else
|
else
|
||||||
for i in pkgver pkgrel epoch; do
|
for i in pkgver pkgrel epoch; do
|
||||||
|
@ -253,9 +253,9 @@ get_full_version() {
|
||||||
[[ -z ${!indirect} ]] && eval ${indirect}=\"${!i}\"
|
[[ -z ${!indirect} ]] && eval ${indirect}=\"${!i}\"
|
||||||
done
|
done
|
||||||
if (( ! $epoch_override )); then
|
if (( ! $epoch_override )); then
|
||||||
echo $pkgver_override-$pkgrel_override
|
printf "%s\n" "$pkgver_override-$pkgrel_override"
|
||||||
else
|
else
|
||||||
echo $epoch_override:$pkgver_override-$pkgrel_override
|
printf "%s\n" "$epoch_override:$pkgver_override-$pkgrel_override"
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
@ -272,14 +272,14 @@ get_full_version() {
|
||||||
check_option() {
|
check_option() {
|
||||||
local ret=$(in_opt_array "$1" ${options[@]})
|
local ret=$(in_opt_array "$1" ${options[@]})
|
||||||
if [[ $ret != '?' ]]; then
|
if [[ $ret != '?' ]]; then
|
||||||
echo $ret
|
printf "%s\n" "$ret"
|
||||||
return
|
return
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# fall back to makepkg.conf options
|
# fall back to makepkg.conf options
|
||||||
ret=$(in_opt_array "$1" ${OPTIONS[@]})
|
ret=$(in_opt_array "$1" ${OPTIONS[@]})
|
||||||
if [[ $ret != '?' ]]; then
|
if [[ $ret != '?' ]]; then
|
||||||
echo $ret
|
printf "%s\n" "$ret"
|
||||||
return
|
return
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
@ -379,7 +379,7 @@ get_downloadclient() {
|
||||||
exit 1 # $E_MISSING_PROGRAM
|
exit 1 # $E_MISSING_PROGRAM
|
||||||
fi
|
fi
|
||||||
|
|
||||||
echo "$agent"
|
printf "%s\n" "$agent"
|
||||||
}
|
}
|
||||||
|
|
||||||
download_file() {
|
download_file() {
|
||||||
|
@ -446,7 +446,7 @@ check_deps() {
|
||||||
set -E
|
set -E
|
||||||
|
|
||||||
if (( ret == 127 )); then #unresolved deps
|
if (( ret == 127 )); then #unresolved deps
|
||||||
echo "$pmout"
|
printf "%s\n" "$pmout"
|
||||||
elif (( ret )); then
|
elif (( ret )); then
|
||||||
error "$(gettext "'%s' returned a fatal error (%i): %s")" "$PACMAN" "$ret" "$pmout"
|
error "$(gettext "'%s' returned a fatal error (%i): %s")" "$PACMAN" "$ret" "$pmout"
|
||||||
return "$ret"
|
return "$ret"
|
||||||
|
@ -594,9 +594,9 @@ get_integlist() {
|
||||||
done
|
done
|
||||||
|
|
||||||
if (( ${#integlist[@]} > 0 )); then
|
if (( ${#integlist[@]} > 0 )); then
|
||||||
echo ${integlist[@]}
|
printf "%s\n" "${integlist[@]}"
|
||||||
else
|
else
|
||||||
echo ${INTEGRITY_CHECK[@]}
|
printf "%s\n" "${INTEGRITY_CHECK[@]}"
|
||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -627,7 +627,7 @@ generate_checksums() {
|
||||||
|
|
||||||
local ct=0
|
local ct=0
|
||||||
local numsrc=${#source[@]}
|
local numsrc=${#source[@]}
|
||||||
echo -n "${integ}sums=("
|
printf "%s" "${integ}sums=("
|
||||||
|
|
||||||
local i
|
local i
|
||||||
local indent=''
|
local indent=''
|
||||||
|
@ -641,8 +641,8 @@ generate_checksums() {
|
||||||
file="$(get_filepath "$netfile")" || missing_source_file "$netfile"
|
file="$(get_filepath "$netfile")" || missing_source_file "$netfile"
|
||||||
local sum="$(openssl dgst -${integ} "$file")"
|
local sum="$(openssl dgst -${integ} "$file")"
|
||||||
sum=${sum##* }
|
sum=${sum##* }
|
||||||
(( ct )) && echo -n "$indent"
|
(( ct )) && printf "%s" "$indent"
|
||||||
echo -n "'$sum'"
|
printf "%s" "'$sum'"
|
||||||
ct=$(($ct+1))
|
ct=$(($ct+1))
|
||||||
(( $ct < $numsrc )) && echo
|
(( $ct < $numsrc )) && echo
|
||||||
done
|
done
|
||||||
|
@ -668,7 +668,7 @@ check_checksums() {
|
||||||
for file in "${source[@]}"; do
|
for file in "${source[@]}"; do
|
||||||
local found=1
|
local found=1
|
||||||
file="$(get_filename "$file")"
|
file="$(get_filename "$file")"
|
||||||
echo -n " $file ... " >&2
|
printf "%s" " $file ... " >&2
|
||||||
|
|
||||||
if ! file="$(get_filepath "$file")"; then
|
if ! file="$(get_filepath "$file")"; then
|
||||||
printf -- "$(gettext "NOT FOUND")\n" >&2
|
printf -- "$(gettext "NOT FOUND")\n" >&2
|
||||||
|
@ -1082,7 +1082,7 @@ find_libdepends() {
|
||||||
if in_array "${soname}" ${depends[@]}; then
|
if in_array "${soname}" ${depends[@]}; then
|
||||||
if ! in_array "${soname}=${soversion}-${soarch}" ${libdepends[@]}; then
|
if ! in_array "${soname}=${soversion}-${soarch}" ${libdepends[@]}; then
|
||||||
# libfoo.so=1-64
|
# libfoo.so=1-64
|
||||||
echo "${soname}=${soversion}-${soarch}"
|
printf "%s" "${soname}=${soversion}-${soarch}"
|
||||||
libdepends=(${libdepends[@]} "${soname}=${soversion}-${soarch}")
|
libdepends=(${libdepends[@]} "${soname}=${soversion}-${soarch}")
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
|
@ -1137,7 +1137,7 @@ find_libprovides() {
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
|
|
||||||
echo ${libprovides[@]}
|
printf "%s" "${libprovides[@]}"
|
||||||
}
|
}
|
||||||
|
|
||||||
check_license() {
|
check_license() {
|
||||||
|
@ -1171,15 +1171,15 @@ write_pkginfo() {
|
||||||
echo "# using $(fakeroot -v)"
|
echo "# using $(fakeroot -v)"
|
||||||
fi
|
fi
|
||||||
echo "# $(LC_ALL=C date -u)"
|
echo "# $(LC_ALL=C date -u)"
|
||||||
echo "pkgname = $1"
|
printf "pkgname = %s\n" "$1"
|
||||||
(( SPLITPKG )) && echo pkgbase = $pkgbase
|
(( SPLITPKG )) && echo pkgbase = $pkgbase
|
||||||
echo "pkgver = $(get_full_version)"
|
echo "pkgver = $(get_full_version)"
|
||||||
echo "pkgdesc = $pkgdesc"
|
printf "pkgdesc = %s\n" "$pkgdesc"
|
||||||
echo "url = $url"
|
printf "url = %s\n" "$url"
|
||||||
echo "builddate = $builddate"
|
printf "builddate = %s\n" "$builddate"
|
||||||
echo "packager = $packager"
|
printf "packager = %s\n" "$packager"
|
||||||
echo "size = $size"
|
printf "size = %s\n" "$size"
|
||||||
echo "arch = $PKGARCH"
|
printf "arch = %s\n" "$PKGARCH"
|
||||||
|
|
||||||
[[ $license ]] && printf "license = %s\n" "${license[@]}"
|
[[ $license ]] && printf "license = %s\n" "${license[@]}"
|
||||||
[[ $replaces ]] && printf "replaces = %s\n" "${replaces[@]}"
|
[[ $replaces ]] && printf "replaces = %s\n" "${replaces[@]}"
|
||||||
|
@ -1207,7 +1207,7 @@ write_pkginfo() {
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
else
|
else
|
||||||
echo "depend = $it"
|
printf "depend = %s\n" "$it"
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
|
|
||||||
|
@ -1215,9 +1215,9 @@ write_pkginfo() {
|
||||||
local ret="$(check_option $it)"
|
local ret="$(check_option $it)"
|
||||||
if [[ $ret != "?" ]]; then
|
if [[ $ret != "?" ]]; then
|
||||||
if [[ $ret = "y" ]]; then
|
if [[ $ret = "y" ]]; then
|
||||||
echo "makepkgopt = $it"
|
printf "makepkgopt = %s\n" "$it"
|
||||||
else
|
else
|
||||||
echo "makepkgopt = !$it"
|
printf "makepkgopt = %s\n" "!$it"
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
|
@ -1859,7 +1859,7 @@ canonicalize_path() {
|
||||||
pwd -P
|
pwd -P
|
||||||
)
|
)
|
||||||
else
|
else
|
||||||
echo "$path"
|
printf "%s\n" "$path"
|
||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1924,7 +1924,7 @@ There is NO WARRANTY, to the extent permitted by law.\n")"
|
||||||
# determine whether we have gettext; make it a no-op if we do not
|
# determine whether we have gettext; make it a no-op if we do not
|
||||||
if ! type -p gettext >/dev/null; then
|
if ! type -p gettext >/dev/null; then
|
||||||
gettext() {
|
gettext() {
|
||||||
echo "$@"
|
printf "%s\n" "$@"
|
||||||
}
|
}
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue