From 499931bffdd1934d022371460c8efd84556c89c1 Mon Sep 17 00:00:00 2001 From: Gerhard Sittig Date: Sun, 21 May 2017 18:18:43 +0200 Subject: [PATCH] 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). --- source/new-driver | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) 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()), -- 2.30.2