From 02c4e76c642f4de4820a1e07620cdd7ba5fff18d Mon Sep 17 00:00:00 2001
From: Martin Pitt <martin.pitt@canonical.com>
Date: Mon, 26 Oct 2015 09:19:05 +0100
Subject: [PATCH] run-autopkgtest: Require --trigger argument, improve help

---
 run-autopkgtest | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/run-autopkgtest b/run-autopkgtest
index 642d160..f1a02f6 100755
--- a/run-autopkgtest
+++ b/run-autopkgtest
@@ -24,13 +24,23 @@ def parse_args():
                         help='Only run test(s) on given architecture name(s). '
                         'Can be specified multiple times (default: all).')
     parser.add_argument('--trigger', action='append', default=[],
+                        metavar='SOURCE/VERSION', required=True,
                         help='Add triggering package to request. '
                         'Can be specified multiple times.')
     parser.add_argument('--ppa', metavar='LPUSER/PPANAME',
                         help='Enable PPA for requested test(s)')
     parser.add_argument('package', nargs='+',
                         help='Source package name(s) whose tests to run.')
-    return parser.parse_args()
+    args = parser.parse_args()
+
+    # verify syntax of triggers
+    for t in args.trigger:
+        try:
+            (src, ver) = t.split('/')
+        except ValueError:
+            parser.error('Invalid trigger format "%s", must be "sourcepkg/version"' % t)
+
+    return args
 
 
 def parse_config(config_file):