diff options
author | Nirbheek Chauhan <nirbheek@centricular.com> | 2020-02-21 05:09:05 +0530 |
---|---|---|
committer | Nirbheek Chauhan <nirbheek@centricular.com> | 2020-02-21 05:45:34 +0530 |
commit | 36b4dec26270e8908307e80fcc174004bc69a181 (patch) | |
tree | ec76ce18bdad8cc8149a14b436a56a2506ae3a84 | |
parent | 090eaac91881b3cff50369fa0b1d53dcd5167fe6 (diff) | |
download | meson-nirbheek/strip-more-unused-linkerlike-args.tar.gz |
macOS: Remove more unused linkerlike argsnirbheek/strip-more-unused-linkerlike-args
`-L` and `-headerpad_max_install_names` are both linker arguments that
are commonly passed in CFLAGS too.
Closes https://github.com/mesonbuild/meson/issues/6294
-rw-r--r-- | mesonbuild/compilers/compilers.py | 20 | ||||
-rwxr-xr-x | run_unittests.py | 5 |
2 files changed, 24 insertions, 1 deletions
diff --git a/mesonbuild/compilers/compilers.py b/mesonbuild/compilers/compilers.py index e8e72cfaf..03b1a3338 100644 --- a/mesonbuild/compilers/compilers.py +++ b/mesonbuild/compilers/compilers.py @@ -1059,7 +1059,25 @@ class Compiler: return self.linker.get_undefined_link_args() def remove_linkerlike_args(self, args): - return [x for x in args if not x.startswith('-Wl')] + rm_exact = ('-headerpad_max_install_names',) + rm_prefixes = ('-Wl,', '-L',) + rm_next = ('-L',) + ret = [] + iargs = iter(args) + for arg in iargs: + # Remove this argument + if arg in rm_exact: + continue + # If the argument starts with this, but is not *exactly* this + # f.ex., '-L' should match ['-Lfoo'] but not ['-L', 'foo'] + if arg.startswith(rm_prefixes) and arg not in rm_prefixes: + continue + # Ignore this argument and the one after it + if arg in rm_next: + next(iargs) + continue + ret.append(arg) + return ret def get_lto_compile_args(self) -> T.List[str]: return [] diff --git a/run_unittests.py b/run_unittests.py index 5bded1703..56618df09 100755 --- a/run_unittests.py +++ b/run_unittests.py @@ -4896,6 +4896,11 @@ class DarwinTests(BasePlatformTests): self.build() self.install() + def test_removing_unused_linker_args(self): + testdir = os.path.join(self.common_test_dir, '108 has arg') + env = {'CFLAGS': '-L/tmp -L /var/tmp -headerpad_max_install_names -Wl,-export_dynamic'} + self.init(testdir, override_envvars=env) + @unittest.skipUnless(not is_windows(), "requires something Unix-like") class LinuxlikeTests(BasePlatformTests): |