[pacman-dev] [PATCH 2/2] makepkg: avoid unnecessary local variable

[Dave Reisner]

Use a simple default value parameter expansion instead of using an
intermediate local variable to write the packager name in write_pkginfo.

Signed-off-by: Dave Reisner <dreis...@archlinux.org>
---
 scripts/makepkg.sh.in |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index c2941b2..4be17e5 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -1151,11 +1151,6 @@ check_license() {
 
 write_pkginfo() {
        local builddate=$(date -u "+%s")
-       if [[ -n $PACKAGER ]]; then
-               local packager="$PACKAGER"
-       else
-               local packager="Unknown Packager"
-       fi
        local size="$(@DUPATH@ -sk)"
        size="$(( ${size%%[^0-9]*} * 1024 ))"
 
@@ -1171,7 +1166,7 @@ write_pkginfo() {
        echo "pkgdesc = $pkgdesc"
        echo "url = $url"
        echo "builddate = $builddate"
-       echo "packager = $packager"
+       echo "packager = ${PACKAGER:-Unknown Packager}"
        echo "size = $size"
        echo "arch = $PKGARCH"
 
-- 
1.7.9.4