Skip to content

Commit

Permalink
bug 1392603 - Remove toolkit/crashreporter/tools/win32. r=gsvelto
Browse files Browse the repository at this point in the history
We have prebuilt dump_syms binaries in toolkit/crashreporter/tools/win32 from
the days when we didn't build it as part of the build. There's one per MSVC
version, since the APIs it uses are version-specific, but we don't actually
support building with the MSVC versions corresponding to the binaries in that
directory anymore, so we should just remove them.

The only code change necessary here is changing the functional test to skip
running the test if a dump_syms binary isn't available.

MozReview-Commit-ID: IBuJZMVfv7C

Differential Revision: https://phabricator.services.mozilla.com/D15717

--HG--
extra : moz-landing-system : lando
  • Loading branch information
luser committed Jan 4, 2019
1 parent 12c357c commit 96349b5
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions toolkit/crashreporter/tools/unit-symbolstore.py
Original file line number Diff line number Diff line change
Expand Up @@ -479,12 +479,7 @@ def setUp(self):
'dist', 'host', 'bin',
'dump_syms.exe')
else:
self.dump_syms = os.path.join(self.topsrcdir,
'toolkit',
'crashreporter',
'tools',
'win32',
'dump_syms_vc{_MSC_VER}.exe'.format(**buildconfig.substs))
self.skip_test = True
self.target_bin = os.path.join(buildconfig.topobjdir,
'dist', 'bin',
'firefox.exe')
Expand Down
Binary file not shown.
Binary file not shown.
Binary file not shown.

0 comments on commit 96349b5

Please sign in to comment.