diff options
author | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2014-02-17 15:36:56 +0000 |
---|---|---|
committer | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2014-03-04 13:17:06 +0000 |
commit | d2168a0fe0bc7f05caaeaf81681d0c3d17731e14 (patch) | |
tree | 6f8ea1e234ddc0b644aa35c72d79e88d0222e655 /morphlib/writeexts.py | |
parent | 976db9042d851de2c70e2719c32f611d68681209 (diff) | |
download | morph-d2168a0fe0bc7f05caaeaf81681d0c3d17731e14.tar.gz |
Make parse_autostart() into more general get_environment_boolean()
Also, be more flexible when parsing environment booleans -- convert to
lower case and match 0/1 and true/false as well as yes/no.
Diffstat (limited to 'morphlib/writeexts.py')
-rw-r--r-- | morphlib/writeexts.py | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/morphlib/writeexts.py b/morphlib/writeexts.py index a07c697f..1849f406 100644 --- a/morphlib/writeexts.py +++ b/morphlib/writeexts.py @@ -402,14 +402,14 @@ class WriteExtension(cliapp.Application): return value == 'yes' - def parse_autostart(self): - '''Parse $AUTOSTART to determine if VMs should be started.''' + def get_environment_boolean(self, variable): + '''Parse a yes/no boolean passed through the environment.''' - autostart = os.environ.get('AUTOSTART', 'no') - if autostart == 'no': + value = os.environ.get(variable, 'no').lower() + if value in ['no', '0', 'false']: return False - elif autostart == 'yes': + elif value in ['yes', '1', 'true']: return True else: - raise cliapp.AppException('Unexpected value for AUTOSTART: %s' % - autostart) + raise cliapp.AppException('Unexpected value for %s: %s' % + (variable, value)) |