diff options
author | Aleksey Filippov <alekseyf@google.com> | 2018-03-25 01:19:39 +0000 |
---|---|---|
committer | Aleksey Filippov <alekseyf@google.com> | 2018-03-25 01:33:33 +0000 |
commit | 642df0505adaa0ba3f4b964665887278df099629 (patch) | |
tree | 8c0d8d266c071c0000cf447760d9b95b785d4ab6 /mesonbuild/interpreter.py | |
parent | 708e4b1258d3cdad5c88fa31a6c2fa0890a8415c (diff) | |
download | meson-642df0505adaa0ba3f4b964665887278df099629.zip meson-642df0505adaa0ba3f4b964665887278df099629.tar.gz meson-642df0505adaa0ba3f4b964665887278df099629.tar.bz2 |
Use unholder=True instead of checking held_object attribute presence
Diffstat (limited to 'mesonbuild/interpreter.py')
-rw-r--r-- | mesonbuild/interpreter.py | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/mesonbuild/interpreter.py b/mesonbuild/interpreter.py index 16798a4..13cfb2e 100644 --- a/mesonbuild/interpreter.py +++ b/mesonbuild/interpreter.py @@ -2624,7 +2624,7 @@ root and issuing %s. if 'command' not in kwargs: raise InterpreterException('Missing "command" keyword argument') all_args = extract_as_list(kwargs, 'command') - deps = extract_as_list(kwargs, 'depends') + deps = extract_as_list(kwargs, 'depends', unholder=True) else: raise InterpreterException('Run_target needs at least one positional argument.') @@ -2639,10 +2639,6 @@ root and issuing %s. raise InterpreterException('First argument must be a string.') cleaned_deps = [] for d in deps: - try: - d = d.held_object - except AttributeError: - pass if not isinstance(d, (build.BuildTarget, build.CustomTarget)): raise InterpreterException('Depends items must be build targets.') cleaned_deps.append(d) @@ -3032,11 +3028,9 @@ different subdirectory. if ":" not in setup_name: setup_name = (self.subproject if self.subproject else self.build.project_name) + ":" + setup_name try: - inp = extract_as_list(kwargs, 'exe_wrapper') + inp = extract_as_list(kwargs, 'exe_wrapper', unholder=True) exe_wrapper = [] for i in inp: - if hasattr(i, 'held_object'): - i = i.held_object if isinstance(i, str): exe_wrapper.append(i) elif isinstance(i, dependencies.ExternalProgram): |