diff options
| author | Eli Schwartz <eschwartz@archlinux.org> | 2022-05-29 16:29:17 -0400 |
|---|---|---|
| committer | Xavier Claessens <xclaesse@gmail.com> | 2022-05-30 12:26:19 -0400 |
| commit | 194c28297f3c5a75c4b9918b78742aaabb0a6703 (patch) | |
| tree | 25077e1f87f3a519e6356fb3741494490a6d9141 /mesonbuild/build.py | |
| parent | 915a468ed32feb5bed2bd4e60fefc5754e961a6c (diff) | |
| download | meson-194c28297f3c5a75c4b9918b78742aaabb0a6703.zip meson-194c28297f3c5a75c4b9918b78742aaabb0a6703.tar.gz meson-194c28297f3c5a75c4b9918b78742aaabb0a6703.tar.bz2 | |
fix incorrectly allowed kwarg for custom_target
override_options makes no sense for custom_target as we don't use it for
anything. Also, this was added in commit c3c30d4b060239654c9b848092692ab346ebed9d
despite not being allowed in permittedKwargsc3c30d4b0.
For inexplicable reasons, we had a known_kwargs for custom_target that
looped over kwargs and issued a warning, not an error, for unknown
kwargs. It was impossible to ever hit that check to begin with, though,
ever since commit e08d73510552fa4a3a087af1a9e5fded8f5749fd which added
permittedKwargs and obsoleted those manual checks with real errors.
So at one point override_options was specially permitted to be used
without emitting a warning, and then for about half a decade it was an
error, and then based on some dead code it was allowed again for a bit.
But through all this it doesn't do anything and isn't documented.
Diffstat (limited to 'mesonbuild/build.py')
| -rw-r--r-- | mesonbuild/build.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/mesonbuild/build.py b/mesonbuild/build.py index d64b772..f47279e 100644 --- a/mesonbuild/build.py +++ b/mesonbuild/build.py @@ -2424,7 +2424,6 @@ class CustomTarget(Target, CommandBase): install_dir: T.Optional[T.Sequence[T.Union[str, bool]]] = None, install_mode: T.Optional[FileMode] = None, install_tag: T.Optional[T.Sequence[T.Optional[str]]] = None, - override_options: T.Optional[T.Dict[OptionKey, str]] = None, absolute_paths: bool = False, backend: T.Optional['Backend'] = None, ): @@ -2456,8 +2455,6 @@ class CustomTarget(Target, CommandBase): self.install_tag = _install_tag self.name = name if name else self.outputs[0] - self.set_option_overrides(override_options or {}) - # Whether to use absolute paths for all files on the commandline self.absolute_paths = absolute_paths |