A couple of fixes to the init command's -p option.

Adds windows as one of the allowed platforms flags.
Fixes -p foo to append 'platform-foo', instead of each letter (list.extend
expects a list and thus appends each char in the string, rather than the
string itself).

Change-Id: I73a92127ac29a32fc31b335cc54a246302904140
This commit is contained in:
Pascal Muetschard 2015-10-22 13:26:36 -07:00
parent 745b4ad660
commit c2a64ddffd

View File

@ -179,7 +179,7 @@ to update the working directory files.
r.Save() r.Save()
groups = re.split(r'[,\s]+', opt.groups) groups = re.split(r'[,\s]+', opt.groups)
all_platforms = ['linux', 'darwin'] all_platforms = ['linux', 'darwin', 'windows']
platformize = lambda x: 'platform-' + x platformize = lambda x: 'platform-' + x
if opt.platform == 'auto': if opt.platform == 'auto':
if (not opt.mirror and if (not opt.mirror and
@ -188,7 +188,7 @@ to update the working directory files.
elif opt.platform == 'all': elif opt.platform == 'all':
groups.extend(map(platformize, all_platforms)) groups.extend(map(platformize, all_platforms))
elif opt.platform in all_platforms: elif opt.platform in all_platforms:
groups.extend(platformize(opt.platform)) groups.append(platformize(opt.platform))
elif opt.platform != 'none': elif opt.platform != 'none':
print('fatal: invalid platform flag', file=sys.stderr) print('fatal: invalid platform flag', file=sys.stderr)
sys.exit(1) sys.exit(1)