diff options
author | Luke Shumaker <lukeshu@lukeshu.com> | 2017-04-20 23:05:01 -0400 |
---|---|---|
committer | Luke Shumaker <lukeshu@lukeshu.com> | 2017-04-20 23:19:37 -0400 |
commit | cdc456c8b7c6e3eb4b11d668fb9c6e9bdec7230e (patch) | |
tree | d54d0c816a146023fbc7a54c0d5f71ecd0e69937 /src/xbs-abslibre/helper-abslibre | |
parent | 23213bda24af601acbbea5731246a055680b48d0 (diff) |
Don't use subshells in local/export/declare commands.
Only make this change in places where it shouldn't make a difference, and
something weird has to be going on for the subshell to fail. This is on
par with checking the return value of malloc. We don't need tests for each
of these failure cases.
Diffstat (limited to 'src/xbs-abslibre/helper-abslibre')
-rwxr-xr-x | src/xbs-abslibre/helper-abslibre | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/xbs-abslibre/helper-abslibre b/src/xbs-abslibre/helper-abslibre index 10da489..eab37cd 100755 --- a/src/xbs-abslibre/helper-abslibre +++ b/src/xbs-abslibre/helper-abslibre @@ -1,6 +1,6 @@ #!/usr/bin/env bash -# Copyright (C) 2012-2016 Luke Shumaker <lukeshu@sbcglobal.net> +# Copyright (C) 2012-2017 Luke Shumaker <lukeshu@sbcglobal.net> # # License: GNU GPLv2+ # @@ -83,8 +83,9 @@ release-client() { local arch=$2 conf ABSLIBREDEST - local pkgbase="$(load_PKGBUILD >/dev/null; printf '%s\n' "${pkgbase:-${pkgname}}")" - local pkgdir="${ABSLIBREDEST}/${arch}/${repo}/${pkgbase}" + local pkgbase pkgdir + pkgbase="$(load_PKGBUILD >/dev/null; printf '%s\n' "${pkgbase:-${pkgname}}")" + pkgdir="${ABSLIBREDEST}/${arch}/${repo}/${pkgbase}" lockarch "$arch" if [[ -e $pkgdir ]]; then @@ -103,8 +104,9 @@ release-server() { local arch=$2 conf ABSLIBREDEST - local pkgbase="$(load_PKGBUILD >/dev/null; printf '%s\n' "${pkgbase:-${pkgname}}")" - local pkgdir="${ABSLIBREDEST}/${arch}/${repo}/${pkgbase}" + local pkgbase pkgdir + pkgbase="$(load_PKGBUILD >/dev/null; printf '%s\n' "${pkgbase:-${pkgname}}")" + pkgdir="${ABSLIBREDEST}/${arch}/${repo}/${pkgbase}" lockarch "$arch" checkgit |