From 8036060e1f0f738d4b1c9e7491085fd5adeb41a2 Mon Sep 17 00:00:00 2001
From: Dave Reisner <dreisner@archlinux.org>
Date: Fri, 31 Mar 2017 10:07:51 -0400
Subject: assorted cleanup

mostly to make shellcheck happy
---
 package.inc.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/package.inc.sh b/package.inc.sh
index a0515c9..b1d438b 100644
--- a/package.inc.sh
+++ b/package.inc.sh
@@ -1,7 +1,7 @@
 package_resolve() {
   local pkgbase
 
-  [[ -v pkgname ]] || log_fatal 'BUG: package_resolve called without pkgname var set'
+  [[ $pkgname ]] || log_fatal 'BUG: package_resolve called without pkgname var set'
 
   if package_find_remote "$1" "$2"; then
     return 0
@@ -9,7 +9,7 @@ package_resolve() {
 
   if pkgbase=$(archweb_get_pkgbase "$1") && package_find_remote "$pkgbase" "$2"; then
     log_info '%s is part of package %s' "$1" "$pkgbase"
-    printf -v pkgname %s "$pkgbase"
+    pkgname=$pkgbase
     return 0
   fi
 
@@ -64,10 +64,10 @@ package_log() {
 
   case $method in
     shortlog)
-      logargs=(--pretty=oneline)
+      logargs=('--pretty=oneline')
       ;;
     difflog)
-      logargs=(-p)
+      logargs=('-p')
       ;;
     log)
       logargs=()
-- 
cgit v1.2.3-54-g00ecf