diff options
| author | Benjamin Schubert <ben.c.schubert@gmail.com> | 2019-07-16 18:31:19 +0100 |
|---|---|---|
| committer | bst-marge-bot <marge-bot@buildstream.build> | 2019-07-17 08:26:26 +0000 |
| commit | b2a91466511f60e33e2e77fed2bf9c60bc31803d (patch) | |
| tree | a34e90c980bf95d96c5db60a822e8954edf5c150 /src/buildstream/_project.py | |
| parent | a7ac4fed521881f54b76b199233c5aee3ba1f1ce (diff) | |
| download | buildstream-b2a91466511f60e33e2e77fed2bf9c60bc31803d.tar.gz | |
node: Add 'get_str_list' on 'MappingNode'
`mapping.get_sequence(...).as_str_list()` is a very common
pattern seen both in plugins and the core. Adding a helper to reduce
the number of operations will make usage smoother
Diffstat (limited to 'src/buildstream/_project.py')
| -rw-r--r-- | src/buildstream/_project.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/buildstream/_project.py b/src/buildstream/_project.py index 6a2c0f347..95afc78b5 100644 --- a/src/buildstream/_project.py +++ b/src/buildstream/_project.py @@ -591,10 +591,10 @@ class Project(): defaults = pre_config_node.get_mapping('defaults') defaults.validate_keys(['targets']) - self._default_targets = defaults.get_sequence("targets").as_str_list() + self._default_targets = defaults.get_str_list("targets") # Fatal warnings - self._fatal_warnings = pre_config_node.get_sequence('fatal-warnings', default=[]).as_str_list() + self._fatal_warnings = pre_config_node.get_str_list('fatal-warnings', default=[]) self.loader = Loader(self._context, self, parent=parent_loader, fetch_subprojects=fetch_subprojects) @@ -668,7 +668,7 @@ class Project(): # Load sandbox environment variables self.base_environment = config.get_mapping('environment') - self.base_env_nocache = config.get_sequence('environment-nocache').as_str_list() + self.base_env_nocache = config.get_str_list('environment-nocache') # Load sandbox configuration self._sandbox = config.get_mapping('sandbox') @@ -700,7 +700,7 @@ class Project(): # Parse shell options shell_options = config.get_mapping('shell') shell_options.validate_keys(['command', 'environment', 'host-files']) - self._shell_command = shell_options.get_sequence('command').as_str_list() + self._shell_command = shell_options.get_str_list('command') # Perform environment expansion right away shell_environment = shell_options.get_mapping('environment', default={}) |