diff options
| author | Nirbheek Chauhan <nirbheek@centricular.com> | 2018-05-30 15:00:37 +0530 |
|---|---|---|
| committer | Nirbheek Chauhan <nirbheek@centricular.com> | 2018-05-30 15:00:37 +0530 |
| commit | add47c1408387fda2d1b5e991e4c98244c93511b (patch) | |
| tree | 15c0b701a8f959cd9da6520359234c8a20dbd088 | |
| parent | c87c42b736197b726f3cca47e92bc836c773085e (diff) | |
| download | meson-nirbheek/name_suffix-empty-string.tar.gz | |
Don't accept an empty string for name_suffix:nirbheek/name_suffix-empty-string
This is never going to be useful, and the error message now points
people to what they might be expecting: use the default value for this
platform.
| -rw-r--r-- | mesonbuild/build.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/mesonbuild/build.py b/mesonbuild/build.py index 362a6de06..b6399bbad 100644 --- a/mesonbuild/build.py +++ b/mesonbuild/build.py @@ -789,6 +789,10 @@ This will become a hard error in a future Meson release.''') else: if not isinstance(name_suffix, str): raise InvalidArguments('name_suffix must be a string.') + if name_suffix == '': + raise InvalidArguments('name_suffix should not be an empty string. ' + 'If you want meson to use the default behaviour ' + 'for each platform pass `[]` (empty array)') self.suffix = name_suffix self.name_suffix_set = True if isinstance(self, StaticLibrary): |