aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNirbheek Chauhan <nirbheek@centricular.com>2017-03-03 15:41:34 +0530
committerNirbheek Chauhan <nirbheek@centricular.com>2017-03-03 15:52:31 +0530
commit303b8819ecc4adf5268d0a73aa25aa1540b3c2bc (patch)
treebf6d2eccb6a4e5eb44add68a19d2dbc12a96dc5e
parent2340fd3d8aa5c6ed829ff3bda8a64137ead4b8eb (diff)
downloadmeson-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.py6
-rw-r--r--mesonbuild/backend/ninjabackend.py7
-rw-r--r--mesonbuild/build.py35
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