Browse Source

build: upload sentry src bundles on windows as well (#24773)

Samuel Attard 4 years ago
parent
commit
06cb550c75
2 changed files with 13 additions and 4 deletions
  1. 0 2
      .circleci/config.yml
  2. 13 2
      script/release/uploaders/upload-symbols.py

+ 0 - 2
.circleci/config.yml

@@ -745,8 +745,6 @@ step-maybe-generate-breakpad-symbols: &step-maybe-generate-breakpad-symbols
       if [ "$GENERATE_SYMBOLS" == "true" ]; then
         cd src
         ninja -C out/Default electron:electron_symbols
-        cd out/Default/breakpad_symbols
-        find . -name \*.sym -print0 | xargs -0 npx @sentry/[email protected] difutil bundle-sources
       fi
 
 step-maybe-zip-symbols: &step-maybe-zip-symbols

+ 13 - 2
script/release/uploaders/upload-symbols.py

@@ -1,7 +1,8 @@
 #!/usr/bin/env python
 
-import os
 import glob
+import os
+import subprocess
 import sys
 
 sys.path.append(
@@ -22,6 +23,10 @@ PDB_LIST = [
   os.path.join(RELEASE_DIR, '{0}.exe.pdb'.format(PROJECT_NAME))
 ]
 
+NPX_CMD = "npx"
+if sys.platform == "win32":
+    NPX_CMD += ".cmd"
+
 
 def main():
   os.chdir(ELECTRON_DIR)
@@ -30,7 +35,13 @@ def main():
       run_symstore(pdb, SYMBOLS_DIR, PRODUCT_NAME)
     files = glob.glob(SYMBOLS_DIR + '/*.pdb/*/*.pdb')
   else:
-    files = glob.glob(SYMBOLS_DIR + '/*/*/*.sym') + glob.glob(SYMBOLS_DIR + '/*/*/*.src.zip')
+    files = glob.glob(SYMBOLS_DIR + '/*/*/*.sym')
+
+  for symbol_file in files:
+    print("Generating Sentry src bundle for: " + symbol_file)
+    subprocess.check_output([NPX_CMD, '@sentry/[email protected]', 'difutil', 'bundle-sources', symbol_file])
+
+  files += glob.glob(SYMBOLS_DIR + '/*/*/*.src.zip')
 
   # The file upload needs to be atom-shell/symbols/:symbol_name/:hash/:symbol
   os.chdir(SYMBOLS_DIR)