Commit 95075fba authored by Guido Trotter's avatar Guido Trotter
Browse files

Populate OS variants if an api >= 15 is present

Adding the file name to the os_files dict will fill in the full path and
get it checked, if present we also read it and split into lines, one per
declared variant.
Signed-off-by: default avatarGuido Trotter <>
Reviewed-by: default avatarOlivier Tharan <>
parent 6d79896b
......@@ -1697,6 +1697,9 @@ def _TryOSFromDisk(name, base_dir=None):
# OS Files dictionary, we will populate it with the absolute path names
os_files = dict.fromkeys(constants.OS_SCRIPTS)
if max(api_versions) >= constants.OS_API_V15:
os_files[constants.OS_VARIANTS_FILE] = ''
for name in os_files:
os_files[name] = os.path.sep.join([os_dir, name])
......@@ -1715,12 +1718,23 @@ def _TryOSFromDisk(name, base_dir=None):
return False, ("File '%s' under path '%s' is not executable" %
(name, os_dir))
variants = None
if constants.OS_VARIANTS_FILE in os_files:
variants_file = os_files[constants.OS_VARIANTS_FILE]
variants = utils.ReadFile(variants_file).splitlines()
except EnvironmentError, err:
return False, ("Error while reading the OS variants file at %s: %s" %
(variants_file, _ErrnoOrStr(err)))
if not variants:
return False, ("No supported os variant found")
os_obj = objects.OS(name=name, path=os_dir,
return True, os_obj
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment