[arch-projects] [PATCH] [dbscripts 1/2] db-*: always handle pkgname/pkgbase/pkgfile argument last

Dan McGee dan at archlinux.org
Tue Nov 1 10:55:25 EDT 2011


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 at archlinux.org>
---
 db-remove      |    8 ++++----
 db-repo-add    |    8 ++++----
 db-repo-remove |    8 ++++----
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/db-remove b/db-remove
index b44eb33..e7326c0 100755
--- a/db-remove
+++ b/db-remove
@@ -4,13 +4,13 @@
 . "$(dirname $0)/config"
 
 if [ $# -ne 3 ]; then
-	msg "usage: $(basename $0) <pkgname|pkgbase> <repo> <arch>"
+	msg "usage: $(basename $0) <repo> <arch> <pkgname|pkgbase>"
 	exit 1
 fi
 
-pkgbase="$1"
-repo="$2"
-arch="$3"
+repo="$1"
+arch="$2"
+pkgbase="$3"
 
 ftppath="$FTP_BASE/$repo/os"
 svnrepo="$repo-$arch"
diff --git a/db-repo-add b/db-repo-add
index 53cfc84..8d4a56d 100755
--- a/db-repo-add
+++ b/db-repo-add
@@ -4,13 +4,13 @@
 . "$(dirname $0)/config"
 
 if [ $# -ne 3 ]; then
-	msg "usage: $(basename $0) <pkgfile> <repo> <arch>"
+	msg "usage: $(basename $0) <repo> <arch> <pkgfile>"
 	exit 1
 fi
 
-pkgfile="$1"
-repo="$2"
-arch="$3"
+repo="$1"
+arch="$2"
+pkgfile="$3"
 
 ftppath="$FTP_BASE/$repo/os"
 
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"
 
-- 
1.7.7.1



More information about the arch-projects mailing list