Skip to content

Commit

Permalink
build: put v8_context_snapshot_generator.dSYM in its own zip file (#3…
Browse files Browse the repository at this point in the history
…3187)

Co-authored-by: John Kleinschmidt <jkleinsc@electronjs.org>
  • Loading branch information
trop[bot] and jkleinsc committed Mar 8, 2022
1 parent b4466f8 commit 7b76a70
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 1 deletion.
4 changes: 4 additions & 0 deletions script/release/release.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,9 +111,11 @@ function assetsForVersion (version, validatingRelease) {
`chromedriver-${version}-win32-x64.zip`,
`chromedriver-${version}-win32-arm64.zip`,
`electron-${version}-darwin-x64-dsym.zip`,
`electron-${version}-darwin-x64-dsym-snapshot.zip`,
`electron-${version}-darwin-x64-symbols.zip`,
`electron-${version}-darwin-x64.zip`,
`electron-${version}-darwin-arm64-dsym.zip`,
`electron-${version}-darwin-arm64-dsym-snapshot.zip`,
`electron-${version}-darwin-arm64-symbols.zip`,
`electron-${version}-darwin-arm64.zip`,
`electron-${version}-linux-arm64-symbols.zip`,
Expand All @@ -126,9 +128,11 @@ function assetsForVersion (version, validatingRelease) {
`electron-${version}-linux-x64-symbols.zip`,
`electron-${version}-linux-x64.zip`,
`electron-${version}-mas-x64-dsym.zip`,
`electron-${version}-mas-x64-dsym-snapshot.zip`,
`electron-${version}-mas-x64-symbols.zip`,
`electron-${version}-mas-x64.zip`,
`electron-${version}-mas-arm64-dsym.zip`,
`electron-${version}-mas-arm64-dsym-snapshot.zip`,
`electron-${version}-mas-arm64-symbols.zip`,
`electron-${version}-mas-arm64.zip`,
`electron-${version}-win32-ia32-pdb.zip`,
Expand Down
7 changes: 6 additions & 1 deletion script/release/uploaders/upload.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@
DIST_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION)
SYMBOLS_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION, 'symbols')
DSYM_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION, 'dsym')
DSYM_SNAPSHOT_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION,
'dsym-snapshot')
PDB_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION, 'pdb')
DEBUG_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION, 'debug')
TOOLCHAIN_PROFILE_NAME = get_zip_name(PROJECT_NAME, ELECTRON_VERSION,
Expand Down Expand Up @@ -88,6 +90,10 @@ def main():
dsym_zip = os.path.join(OUT_DIR, DSYM_NAME)
shutil.copy2(os.path.join(OUT_DIR, 'dsym.zip'), dsym_zip)
upload_electron(release, dsym_zip, args)

dsym_snaphot_zip = os.path.join(OUT_DIR, DSYM_SNAPSHOT_NAME)
shutil.copy2(os.path.join(OUT_DIR, 'dsym-snapshot.zip'), dsym_snaphot_zip)
upload_electron(release, dsym_snaphot_zip, args)
elif PLATFORM == 'win32':
pdb_zip = os.path.join(OUT_DIR, PDB_NAME)
shutil.copy2(os.path.join(OUT_DIR, 'pdb.zip'), pdb_zip)
Expand Down Expand Up @@ -154,7 +160,6 @@ def main():
'toolchain_profile.json')
upload_electron(release, toolchain_profile_zip, args)


def parse_args():
parser = argparse.ArgumentParser(description='upload distribution file')
parser.add_argument('-v', '--version', help='Specify the version',
Expand Down
8 changes: 8 additions & 0 deletions script/zip-symbols.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,17 @@ def main():
dsym_name = 'dsym.zip'
with scoped_cwd(args.build_dir):
dsyms = glob.glob('*.dSYM')
snapshot_dsyms = ['v8_context_snapshot_generator.dSYM']
for dsym in snapshot_dsyms:
if (dsym in dsyms):
dsyms.remove(dsym)
dsym_zip_file = os.path.join(args.build_dir, dsym_name)
print('Making dsym zip: ' + dsym_zip_file)
make_zip(dsym_zip_file, licenses, dsyms)
dsym_snapshot_name = 'dsym-snapshot.zip'
dsym_snapshot_zip_file = os.path.join(args.build_dir, dsym_snapshot_name)
print('Making dsym snapshot zip: ' + dsym_snapshot_zip_file)
make_zip(dsym_snapshot_zip_file, licenses, snapshot_dsyms)
if len(dsyms) > 0 and 'DELETE_DSYMS_AFTER_ZIP' in os.environ:
execute(['rm', '-rf'] + dsyms)
elif PLATFORM == 'win32':
Expand Down

0 comments on commit 7b76a70

Please sign in to comment.