From aa4b36a03ddcf98874b67d926b861196e07d7df7 Mon Sep 17 00:00:00 2001 From: Robo Date: Wed, 11 Dec 2019 09:42:57 -0800 Subject: [PATCH] fix: avoid Electron.dsym files in the main app bundle (#21447) * ci: CHECK_DIST_MANIFEST in release builds * fix: skip Electron.dSYM on macOS app zip --- .circleci/config.yml | 1 + build/zip.py | 9 ++++++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index 6e8b6d5999bf7..4dc89e61eee31 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -81,6 +81,7 @@ env-release-build: &env-release-build GN_CONFIG: //electron/build/args/release.gn STRIP_BINARIES: true GENERATE_SYMBOLS: true + CHECK_DIST_MANIFEST: '1' env-headless-testing: &env-headless-testing DISPLAY: ':99.0' diff --git a/build/zip.py b/build/zip.py index 00a6bc9de097a..0e5c3697317c0 100644 --- a/build/zip.py +++ b/build/zip.py @@ -16,6 +16,10 @@ './libVkICD_mock_', #Skipping because these are outputs that we don't need './VkICD_mock_', #Skipping because these are outputs that we don't need + # Skipping because its an output of create_bundle from //build/config/mac/rules.gni + # that we don't need + 'Electron.dSYM', + # //chrome/browser:resources depends on this via # //chrome/browser/resources/ssl/ssl_error_assistant, but we don't need to # ship it. @@ -52,14 +56,13 @@ def main(argv): with open(runtime_deps) as f: for dep in f.readlines(): dep = dep.strip() - dist_files.add(dep) + if not skip_path(dep, dist_zip, target_cpu): + dist_files.add(dep) if sys.platform == 'darwin' and not should_flatten: execute(['zip', '-r', '-y', dist_zip] + list(dist_files)) else: with zipfile.ZipFile(dist_zip, 'w', zipfile.ZIP_DEFLATED, allowZip64=True) as z: for dep in dist_files: - if skip_path(dep, dist_zip, target_cpu): - continue if os.path.isdir(dep): for root, dirs, files in os.walk(dep): for file in files: