diff options
author | Nirbheek Chauhan <nirbheek@centricular.com> | 2017-03-03 15:41:34 +0530 |
---|---|---|
committer | Nirbheek Chauhan <nirbheek@centricular.com> | 2017-03-03 15:52:31 +0530 |
commit | 303b8819ecc4adf5268d0a73aa25aa1540b3c2bc (patch) | |
tree | bf6d2eccb6a4e5eb44add68a19d2dbc12a96dc5e | |
parent | 2340fd3d8aa5c6ed829ff3bda8a64137ead4b8eb (diff) | |
download | meson-303b8819ecc4adf5268d0a73aa25aa1540b3c2bc.zip meson-303b8819ecc4adf5268d0a73aa25aa1540b3c2bc.tar.gz meson-303b8819ecc4adf5268d0a73aa25aa1540b3c2bc.tar.bz2 |
Nuke dead code which was used for installing PDB files
We check for the existence of PDB files in the install script, so we
don't need to do all this mucking about here. That's more robust too
because we don't need to parse build arguments in buildtype=plain
and decide if the PDB file would be generated.
-rw-r--r-- | mesonbuild/backend/backends.py | 6 | ||||
-rw-r--r-- | mesonbuild/backend/ninjabackend.py | 7 | ||||
-rw-r--r-- | mesonbuild/build.py | 35 |
3 files changed, 0 insertions, 48 deletions
diff --git a/mesonbuild/backend/backends.py b/mesonbuild/backend/backends.py index 26052d3..5939488 100644 --- a/mesonbuild/backend/backends.py +++ b/mesonbuild/backend/backends.py @@ -115,12 +115,6 @@ class Backend: return os.path.join(self.get_target_dir(target), target.get_filename()) raise AssertionError('BUG: Tried to link to something that\'s not a library') - def get_target_debug_filename(self, target): - fname = target.get_debug_filename() - if not fname: - raise AssertionError("BUG: Tried to generate debug filename when it doesn't exist") - return os.path.join(self.get_target_dir(target), fname) - def get_target_dir(self, target): if self.environment.coredata.get_builtin_option('layout') == 'mirror': dirname = target.get_subdir() diff --git a/mesonbuild/backend/ninjabackend.py b/mesonbuild/backend/ninjabackend.py index 5e137ca..16a31a1 100644 --- a/mesonbuild/backend/ninjabackend.py +++ b/mesonbuild/backend/ninjabackend.py @@ -653,13 +653,6 @@ int dummy; else: # XXX: Add BuildTarget-specific install dir cases here outdir = self.environment.get_libdir() - if isinstance(t, build.SharedLibrary) or isinstance(t, build.Executable): - if t.get_debug_filename(): - # Install the debug symbols file in the same place as - # the target itself. It has no aliases, should not be - # stripped, and doesn't have an install_rpath - i = [self.get_target_debug_filename(t), outdir, {}, False, ''] - d.targets.append(i) if isinstance(t, build.BuildTarget): i = [self.get_target_filename(t), outdir, t.get_aliases(), should_strip, t.install_rpath] diff --git a/mesonbuild/build.py b/mesonbuild/build.py index bdb1dc3..fd01f77 100644 --- a/mesonbuild/build.py +++ b/mesonbuild/build.py @@ -300,8 +300,6 @@ class BuildTarget(Target): self.name_prefix_set = False self.name_suffix_set = False self.filename = 'no_name' - # The file with debugging symbols - self.debug_filename = None self.need_install = False self.pch = {} self.extra_args = {} @@ -686,15 +684,6 @@ class BuildTarget(Target): def get_outputs(self): return [self.filename] - def get_debug_filename(self): - """ - The name of the file that contains debugging symbols for this target - - Returns None if there are no debugging symbols or if they are embedded - in the filename itself - """ - return self.debug_filename - def get_extra_args(self, language): return self.extra_args.get(language, []) @@ -1006,10 +995,6 @@ class Executable(BuildTarget): self.filename = self.name if self.suffix: self.filename += '.' + self.suffix - # See determine_debug_filenames() in build.SharedLibrary - buildtype = environment.coredata.get_builtin_option('buildtype') - if self.get_using_msvc() and buildtype.startswith('debug'): - self.debug_filename = self.prefix + self.name + '.pdb' def type_suffix(self): return "@exe" @@ -1037,10 +1022,6 @@ class StaticLibrary(BuildTarget): else: self.suffix = 'a' self.filename = self.prefix + self.name + '.' + self.suffix - # See determine_debug_filenames() in build.SharedLibrary - buildtype = environment.coredata.get_builtin_option('buildtype') - if self.get_using_msvc() and buildtype.startswith('debug'): - self.debug_filename = self.prefix + self.name + '.pdb' def type_suffix(self): return "@sta" @@ -1066,7 +1047,6 @@ class SharedLibrary(BuildTarget): self.suffix = None self.basic_filename_tpl = '{0.prefix}{0.name}.{0.suffix}' self.determine_filenames(is_cross, environment) - self.determine_debug_filenames(is_cross, environment) def determine_filenames(self, is_cross, env): """ @@ -1156,21 +1136,6 @@ class SharedLibrary(BuildTarget): self.suffix = suffix self.filename = self.filename_tpl.format(self) - def determine_debug_filenames(self, is_cross, env): - """ - Determine the debug filename(s) using the prefix/name/etc detected in - determine_filenames() above. - """ - buildtype = env.coredata.get_builtin_option('buildtype') - if self.get_using_msvc() and buildtype.startswith('debug'): - # Currently we only implement separate debug symbol files for MSVC - # since the toolchain does it for us. Other toolchains embed the - # debugging symbols in the file itself by default. - if self.soversion: - self.debug_filename = '{0.prefix}{0.name}-{0.soversion}.pdb'.format(self) - else: - self.debug_filename = '{0.prefix}{0.name}.pdb'.format(self) - def process_kwargs(self, kwargs, environment): super().process_kwargs(kwargs, environment) # Shared library version |