summaryrefslogtreecommitdiff
path: root/indra/newview
diff options
context:
space:
mode:
authorAndrey Kleshchev <andreykproductengine@lindenlab.com>2020-10-06 01:46:00 +0300
committerAndrey Kleshchev <andreykproductengine@lindenlab.com>2020-10-06 21:59:39 +0300
commitf08421e6512da49c633823a65c459de32aae61de (patch)
treee7d10e561e5f849d2c8b2df78ed36fc9cea6da4c /indra/newview
parentfc17790a35402e25d7790dfed467e9c46b059554 (diff)
SL-14012 Crashes from Win10 are sent with uninformative callstacks
Simplify name generation according to slack discussion, also helps bugsplat with matching files
Diffstat (limited to 'indra/newview')
-rwxr-xr-xindra/newview/viewer_manifest.py13
1 files changed, 11 insertions, 2 deletions
diff --git a/indra/newview/viewer_manifest.py b/indra/newview/viewer_manifest.py
index 90a2af98f7..c6f149b5fe 100755
--- a/indra/newview/viewer_manifest.py
+++ b/indra/newview/viewer_manifest.py
@@ -260,6 +260,9 @@ class ViewerManifest(LLManifest):
app_suffix=self.channel_variant()
return CHANNEL_VENDOR_BASE + ' ' + app_suffix
+ def exec_name(self):
+ return "SecondLifeViewer"
+
def app_name_oneword(self):
return ''.join(self.app_name().split())
@@ -419,10 +422,12 @@ class WindowsManifest(ViewerManifest):
build_data_json_platform = 'win'
def final_exe(self):
- return self.app_name_oneword()+".exe"
+ return self.exec_name()+".exe"
+
+ def final_pdb(self):
+ return self.exec_name()+".pdb"
def finish_build_data_dict(self, build_data_dict):
- #MAINT-7294: Windows exe names depend on channel name, so write that in also
build_data_dict['Executable'] = self.final_exe()
build_data_dict['AppName'] = self.app_name()
return build_data_dict
@@ -487,6 +492,10 @@ class WindowsManifest(ViewerManifest):
if self.is_packaging_viewer():
# Find secondlife-bin.exe in the 'configuration' dir, then rename it to the result of final_exe.
self.path(src='%s/secondlife-bin.exe' % self.args['configuration'], dst=self.final_exe())
+ # Bugsplat for some reason requires a match between user's exe and supplied pdb on Windows 10
+ # this feels wrong and doesn't make any sense, since often we are supposed to match a whole
+ # folder of symbol files to a random exe. But Bugsplat says 'it's expected' so just rename files.
+ self.ccopyfile(src='%s/secondlife-bin.pdb' % self.args['configuration'], dst='%s/%s' % (self.args['configuration'], self.final_pdb()))
with self.prefix(src=os.path.join(pkgdir, "VMP")):
# include the compiled launcher scripts so that it gets included in the file_list