From 22dd30328ec4229321f1f96026704ff3cf1cc621 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Tue, 1 Nov 2011 09:55:25 -0500
Subject: db-*: always handle pkgname/pkgbase/pkgfile argument last

This matches what we do in db-update, and makes the most sense if we
want to expand these commands to take multiple arguments at the same
time. Especially in the case of db-repo-add and db-repo-remove, these
wrapper commands should be no more limiting than repo-add and
repo-remove which always allow multiple arguments and are more efficient
that way.

This patch simply reorders the arguments; a later patch will support
multiple pkgname/pkgbase/pkgfile arguments on the command line.

Although this does break backwards compatibility and requires some
changing of muscle memory, I think it is a worthwhile change to make as
the functionality of these commands becomes more powerful and the order
of arguments becomes the same in all of them.

Signed-off-by: Dan McGee <dan@archlinux.org>
---
 db-repo-remove | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'db-repo-remove')

diff --git a/db-repo-remove b/db-repo-remove
index b01910f..e55cd07 100755
--- a/db-repo-remove
+++ b/db-repo-remove
@@ -4,13 +4,13 @@
 . "$(dirname $0)/config"
 
 if [ $# -ne 3 ]; then
-	msg "usage: $(basename $0) <pkgname> <repo> <arch>"
+	msg "usage: $(basename $0) <repo> <arch> <pkgname>"
 	exit 1
 fi
 
-pkgname="$1"
-repo="$2"
-arch="$3"
+repo="$1"
+arch="$2"
+pkgname="$3"
 
 ftppath="$FTP_BASE/$repo/os"
 
-- 
cgit v1.2.3-2-g168b


From bbf00cc90ed8d1bf3bbed1f78acca71b2d624ee2 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Tue, 1 Nov 2011 09:55:26 -0500
Subject: db-repo-{add, remove}: allow specifying multiple packages

This allows invoking these commands for more than one package at a time
which is incredibly more efficient as the database doesn't need to be
locked, unpacked, modified, changed, and unlocked for every single
passed package name or file if you have more than one.

Signed-off-by: Dan McGee <dan@archlinux.org>
---
 db-repo-remove | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

(limited to 'db-repo-remove')

diff --git a/db-repo-remove b/db-repo-remove
index e55cd07..4f04ed1 100755
--- a/db-repo-remove
+++ b/db-repo-remove
@@ -3,14 +3,14 @@
 . "$(dirname $0)/db-functions"
 . "$(dirname $0)/config"
 
-if [ $# -ne 3 ]; then
-	msg "usage: $(basename $0) <repo> <arch> <pkgname>"
+if [ $# -lt 3 ]; then
+	msg "usage: $(basename $0) <repo> <arch> <pkgname> ..."
 	exit 1
 fi
 
 repo="$1"
 arch="$2"
-pkgname="$3"
+pkgnames=(${@:3})
 
 ftppath="$FTP_BASE/$repo/os"
 
@@ -28,9 +28,10 @@ for tarch in ${tarches[@]}; do
 	repo_lock $repo $tarch || exit 1
 done
 
-msg "Removing $pkgname from [$repo]..."
-
 for tarch in ${tarches[@]}; do
-	arch_repo_remove "${repo}" "${tarch}" ${pkgname}
+	for pkgname in ${pkgnames[@]}; do
+		msg "Removing $pkgname from [$repo]..."
+	done
+	arch_repo_remove "${repo}" "${tarch}" ${pkgnames[@]}
 	repo_unlock $repo $tarch
 done
-- 
cgit v1.2.3-2-g168b