From: Gerhard Sittig <redacted>
Date: Sun, 21 May 2017 16:18:43 +0000 (+0200)
Subject: new-driver: do accept multiple command line words for driver name
X-Git-Url: https://sigrok.org/gitaction?a=commitdiff_plain;h=499931bffdd1934d022371460c8efd84556c89c1;p=sigrok-util.git

new-driver: do accept multiple command line words for driver name

The README specifies that driver names consist of multiple words
(typically vendor and model). Adjust the args parser configuration such
that more than one word gets accepted (the previous implementation did
insist on a single word).
---

diff --git a/source/new-driver b/source/new-driver
index 61a3960..14178d3 100755
--- a/source/new-driver
+++ b/source/new-driver
@@ -178,7 +178,7 @@ if __name__ == '__main__':
     author, email = parse_gitconfig()
 
     parser = ArgumentParser(description='Bootstrap a new sigrok hardware driver')
-    parser.add_argument('name', help='new driver name')
+    parser.add_argument('name', nargs='*', default=[], help='new driver name')
     parser.add_argument('--giturl', default=defaulturl,
                         help='URL of the libsigrok git repository '
                         '(defaults to {0})'.format(defaulturl))
@@ -194,7 +194,9 @@ if __name__ == '__main__':
     if not opts.author or not opts.email:
         parser.error('Please provide your username and email address, '
                      'or set your git configuration up.')
-    name = opts.name
+    name = ' '.join(opts.name)
+    if not name:
+        parser.error('Please provide a driver name.')
     names = {
         'name': name,
         'short': re.sub('[^a-z0-9]', '-', name.lower()),