Merge pull request #131 from andythigpen/yaml-include

Fix issues with YAML include tag.
This commit is contained in:
Paulus Schoutsen 2015-05-16 11:45:42 -07:00
commit 8f51741c65

View File

@ -103,6 +103,19 @@ def load_yaml_config_file(config_path):
""" Parse a YAML configuration file. """
import yaml
def parse(fname):
""" Actually parse the file. """
try:
with open(fname) as conf_file:
# If configuration file is empty YAML returns None
# We convert that to an empty dict
conf_dict = yaml.load(conf_file) or {}
except yaml.YAMLError:
_LOGGER.exception('Error reading YAML configuration file %s',
fname)
raise HomeAssistantError()
return conf_dict
def yaml_include(loader, node):
"""
Loads another YAML file and embeds it using the !include tag.
@ -110,20 +123,12 @@ def load_yaml_config_file(config_path):
Example:
device_tracker: !include device_tracker.yaml
"""
fname = os.path.join(os.path.dirname(config_path), node.value)
return load_yaml_config_file(fname)
fname = os.path.join(os.path.dirname(loader.name), node.value)
return parse(fname)
yaml.add_constructor('!include', yaml_include)
try:
with open(config_path) as conf_file:
# If configuration file is empty YAML returns None
# We convert that to an empty dict
conf_dict = yaml.load(conf_file) or {}
except yaml.YAMLError:
_LOGGER.exception('Error reading YAML configuration file')
raise HomeAssistantError()
conf_dict = parse(config_path)
if not isinstance(conf_dict, dict):
_LOGGER.error(