diff options
| author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2018-04-14 18:14:12 +0900 |
|---|---|---|
| committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2018-04-14 18:14:12 +0900 |
| commit | 3ef81d7f6ed8f7ddae2f6c5b5b84020cabf4922f (patch) | |
| tree | ef8ae086f7cce9db64df267a4cda3c65148b50bf | |
| parent | d45e12f775c5ffc077fc48c90bdbdc43c699b320 (diff) | |
| download | buildstream-3ef81d7f6ed8f7ddae2f6c5b5b84020cabf4922f.tar.gz | |
buildelement.py: Use Element.get_variable() instead of poking at private thing
This is the only place where Element._get_variables() was used, which we'll
now happily remove - also Element implementations in general should not
have to use private API.
| -rw-r--r-- | buildstream/buildelement.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/buildstream/buildelement.py b/buildstream/buildelement.py index 55147774d..5f9a856d1 100644 --- a/buildstream/buildelement.py +++ b/buildstream/buildelement.py @@ -134,8 +134,7 @@ class BuildElement(Element): # cache key, while having the side effect of setting max-jobs to 1, # which is normally automatically resolved and does not effect # the cache key. - variables = self._get_variables() - if self.node_get_member(variables.variables, bool, 'notparallel', False): + if self.get_variable('notparallel'): dictionary['notparallel'] = True return dictionary |