Adapt publish to new format, improve completion

completion/bash-completion

@@ -12,16 +12,6 @@
 #
 #	alias fbuild='fdroid build'
 #	complete -F _fdroid_build fbuild
-#
-# There are also completion function for '-p com.some.app' aliases:
-#
-#	alias fbld='fdroid build -v -l -p'
-#	complete -F _fdroid_build_project fbld
-#
-#	alias fcheckup='fdroid checkupdates -v -p'
-#	complete -F _fdroid_checkupdates_project fcheckup
-#
-# This way, one can simply do 'fbld com.some.app' or 'fcheckup com.some.app'
 
 __fdroid_init() {
 	COMPREPLY=()
@@ -38,21 +28,22 @@ __package() {
 	COMPREPLY=( $( compgen -W "$files" -- $cur ) )
 }
 
-__signed_package() {
-	files=( repo/*.apk )
-	if [ "${files[0]}" == "repo/*.apk" ]; then
-		return
-	fi
-	files=( ${files[@]#repo/} )
+__apk_package() {
+	files=( ${1}/*.apk )
+	[ -f "${files[0]}" ] || return
+
+	files=( ${files[@]#*/} )
 	files=${files[@]%_*}
 	COMPREPLY=( $( compgen -W "$files" -- $cur ) )
 }
 
-__signed_vercode() {
+__apk_vercode() {
 	local p
 	p=${cur:0:-1}
 
-	files=( repo/${p}_*.apk )
+	files=( ${1}/${p}_*.apk )
+	[ -f "${files[0]}" ] || return
+
 	files=( ${files[@]#*_} )
 	files=${files[@]%.apk}
 	COMPREPLY=( $( compgen -P "${p}:" -W "$files" -- $cur ) )
@@ -109,10 +100,10 @@ __complete_install() {
 			__complete_options
 			return 0;;
 		*:)
-			__signed_vercode
+			__apk_vercode repo
 			return 0;;
 		*)
-			__signed_package
+			__apk_package repo
 			return 0;;
 	esac
 }
@@ -130,14 +121,19 @@ __complete_update() {
 }
 
 __complete_publish() {
-	opts="-h -v -p"
-	lopts="--help --verbose --package"
-	case "${prev}" in
-		-p|--package)
-			__package
+	opts="-h -v"
+	lopts="--help --verbose"
+	case "${cur}" in
+		-*)
+			__complete_options
+			return 0;;
+		*:)
+			__apk_vercode unsigned
+			return 0;;
+		*)
+			__apk_package unsigned
 			return 0;;
 	esac
-	__complete_options
 }
 
 __complete_checkupdates() {

fdroidserver/build.py

@@ -876,7 +876,7 @@ def main():
     # Get all apps...
     allapps = metadata.read_metadata(xref=not options.onserver)
 
-    apps = common.read_app_args(args, options, allapps, True)
+    apps = common.read_app_args(args, allapps, True)
     apps = [app for app in apps if (options.force or not app['Disabled']) and
             len(app['Repo Type']) > 0 and len(app['builds']) > 0]
 

fdroidserver/checkupdates.py

@@ -295,7 +295,7 @@ def main():
     # Get all apps...
     allapps = metadata.read_metadata(options.verbose)
 
-    apps = common.read_app_args(args, options, allapps, False)
+    apps = common.read_app_args(args, allapps, False)
 
     if options.gplay:
         for app in apps:

fdroidserver/publish.py

@@ -38,10 +38,9 @@ def main():
 
     # Parse command line...
     parser = OptionParser()
+    parser = OptionParser(usage="Usage: %prog [options] [APPID[:VERCODE] [APPID[:VERCODE] ...]]")
     parser.add_option("-v", "--verbose", action="store_true", default=False,
                       help="Spew out even more information than normal")
-    parser.add_option("-p", "--package", default=None,
-                      help="Publish only the specified package")
     (options, args) = parser.parse_args()
 
     config = common.read_config(options)
@@ -74,9 +73,10 @@ def main():
     # and b) a sane-looking ID that would make its way into the repo.
     # Nonetheless, to be sure, before publishing we check that there are no
     # collisions, and refuse to do any publishing if that's the case...
-    apps = metadata.read_metadata()
+    allapps = metadata.read_metadata()
+    vercodes = common.read_pkg_args(args, True)
     allaliases = []
-    for app in apps:
+    for app in allapps:
         m = md5.new()
         m.update(app['id'])
         keyalias = m.hexdigest()[:8]
@@ -85,19 +85,19 @@ def main():
             sys.exit(1)
         allaliases.append(keyalias)
     if options.verbose:
-        print "{0} apps, {0} key aliases".format(len(apps), len(allaliases))
+        print "{0} apps, {0} key aliases".format(len(allapps), len(allaliases))
 
     # Process any apks that are waiting to be signed...
     for apkfile in sorted(glob.glob(os.path.join(unsigned_dir, '*.apk'))):
 
+        appid, vercode = common.apknameinfo(apkfile)
         apkfilename = os.path.basename(apkfile)
-        i = apkfilename.rfind('_')
-        if i == -1:
-            raise BuildException("Invalid apk name")
-        appid = apkfilename[:i]
-        print "Processing " + appid
-
-        if not options.package or options.package == appid:
+        if vercodes and appid not in vercodes:
+            continue
+        if appid in vercodes and vercodes[appid]:
+            if vercode not in vercodes[appid]:
+                continue
+        print "Processing " + apkfile
 
         # Figure out the key alias name we'll use. Only the first 8
         # characters are significant, so we'll use the first 8 from