$EXT is never used, don't set it. The redundant code came from makepkg, which could probably use this fix too. Also fixed is the vim mode line. bacman uses four space indentation, which is against the project guidelines, but consistency in the same file is more important, IMHO. Signed-off-by: lolilolicon <lolilolicon@gmail.com> --- contrib/bacman.in | 30 +++++++++++------------------- 1 files changed, 11 insertions(+), 19 deletions(-) diff --git a/contrib/bacman.in b/contrib/bacman.in index a28f8e7..ec953ce 100755 --- a/contrib/bacman.in +++ b/contrib/bacman.in @@ -77,14 +77,14 @@ fi # parse value of simple, non-repeating variable assignment conf_var_val() { - local var=${1//\//\\/} - awk ' - /^[ \t]*'"${var}"'[ \t]*=/ { - sub(/[^=]+=[ \t]*/, "") - sub(/[ \t]*$/, "") - print - exit - }' + local var=${1//\//\\/} + awk ' + /^[ \t]*'"${var}"'[ \t]*=/ { + sub(/[^=]+=[ \t]*/, "") + sub(/[ \t]*$/, "") + print + exit + }' } DBPath=$(conf_var_val DBPath < @sysconfdir@/pacman.conf) @@ -282,17 +282,7 @@ chmod 644 "$work_dir"/{.PKGINFO,.CHANGELOG,.INSTALL} 2> /dev/null # echo "Generating the package..." -case "$PKGEXT" in - *tar.gz) EXT=${PKGEXT%.gz} ;; - *tar.bz2) EXT=${PKGEXT%.bz2} ;; - *tar.xz) EXT=${PKGEXT%.xz} ;; - *tar) EXT=${PKGEXT} ;; - *) echo "WARNING: '%s' is not a valid archive extension." \ - "$PKGEXT" ; EXT=$PKGEXT ;; -esac - pkg_file="$pkg_dest/$pkg_namver-$pkg_arch${PKGEXT}" -ret=0 # TODO: Maybe this can be set globally for robustness shopt -s -o pipefail @@ -302,6 +292,8 @@ case "$PKGEXT" in *tar.bz2) bzip2 -c -f ;; *tar.xz) xz -c -z - ;; *tar) cat ;; + *) echo "WARNING: '%s' is not a valid archive extension." \ + "$PKGEXT" >&2; cat ;; esac > "${pkg_file}"; ret=$? if [ $ret -ne 0 ]; then @@ -317,5 +309,5 @@ echo Done exit 0 -# vim: set ts=2 sw=2 noet: +# vim: set ts=4 sw=4 et: -- 1.7.6.4