summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <lfleischer@archlinux.org>2016-02-08 07:39:57 +0100
committerLukas Fleischer <lfleischer@archlinux.org>2016-02-09 21:03:47 +0100
commitdc41a5afa5dbbfbca4a711b9059c4b7dc8cbeb72 (patch)
treefc9c5853acdf4f9894274d33121d5bf448fcafab
parent64072461dfcc74857087b23c9ba7d9812b6afe40 (diff)
downloadaur-dc41a5afa5dbbfbca4a711b9059c4b7dc8cbeb72.tar.gz
aur-dc41a5afa5dbbfbca4a711b9059c4b7dc8cbeb72.tar.xz
git-serve: Change syntax for setting keywords
Instead of `config <pkgbase> keywords <keyword1> <keyword2>...`, the new syntax is `set-keywords <pkgbase> <keyword1> <keyword2>...`. The `config` keyword was rather general and it was not obvious that it could be used to change package base settings. Instead of replacing it with an even more verbose expression, remove that unnecessary level of indirection. Since we do not (and probably never will) support keywords anywhere else, the chance of name conflicts is small. Suggested-by: Dave Reisner <dreisner@archlinux.org> Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
-rwxr-xr-xgit-interface/git-serve.py27
1 files changed, 10 insertions, 17 deletions
diff --git a/git-interface/git-serve.py b/git-interface/git-serve.py
index ff389d26..35c6b3a8 100755
--- a/git-interface/git-serve.py
+++ b/git-interface/git-serve.py
@@ -88,7 +88,7 @@ def create_pkgbase(pkgbase, user):
db.close()
-def pkgbase_config_keywords(pkgbase, keywords):
+def pkgbase_set_keywords(pkgbase, keywords):
pkgbase_id = pkgbase_from_name(pkgbase)
if not pkgbase_id:
die('{:s}: package base not found: {:s}'.format(action, pkgbase))
@@ -173,17 +173,10 @@ if action == 'git-upload-pack' or action == 'git-receive-pack':
os.environ["GIT_NAMESPACE"] = pkgbase
cmd = action + " '" + repo_path + "'"
os.execl(git_shell_cmd, git_shell_cmd, '-c', cmd)
-elif action == 'config':
+elif action == 'set-keywords':
if len(cmdargv) < 2:
die_with_help("{:s}: missing repository name".format(action))
- if len(cmdargv) < 3:
- die_with_help("{:s}: missing option name".format(action))
- pkgbase = cmdargv[1]
- option = cmdargv[2]
-
- if option == 'keywords':
- pkgbase_config_keywords(pkgbase, cmdargv[3:])
-
+ pkgbase_set_keywords(cmdargv[1], cmdargv[2:])
elif action == 'list-repos':
if len(cmdargv) > 1:
die_with_help("{:s}: too many arguments".format(action))
@@ -213,12 +206,12 @@ elif action == 'restore':
os.execl(git_update_cmd, git_update_cmd, 'restore')
elif action == 'help':
die("Commands:\n" +
- " config <name> <param>... Change package base settings.\n" +
- " help Show this help message and exit.\n" +
- " list-repos List all your repositories.\n" +
- " restore <name> Restore a deleted package base.\n" +
- " setup-repo <name> Create an empty repository.\n" +
- " git-receive-pack Internal command used with Git.\n" +
- " git-upload-pack Internal command used with Git.")
+ " help Show this help message and exit.\n" +
+ " list-repos List all your repositories.\n" +
+ " restore <name> Restore a deleted package base.\n" +
+ " set-keywords <name> [...] Change package base keywords.\n" +
+ " setup-repo <name> Create an empty repository.\n" +
+ " git-receive-pack Internal command used with Git.\n" +
+ " git-upload-pack Internal command used with Git.")
else:
die_with_help("invalid command: {:s}".format(action))