From f149686bb9c7249c24543bfd758729e733697bb2 Mon Sep 17 00:00:00 2001 From: Keeley Hammond Date: Mon, 11 Apr 2022 11:40:57 -0700 Subject: [PATCH] build: temporarily disable 32-bit Windows symbol generation (#33653) (#33689) * build: temporarily disable 32-bit Windows symbol generation (#33653) * build: temporarily disable 32-bit Windows symbol generation * fix: modify upload.py * chore: fix comment Co-authored-by: VerteDinde * chore: upload correct assets Co-authored-by: John Kleinschmidt --- appveyor.yml | 9 ++++++--- script/release/release.js | 5 +++-- script/release/uploaders/upload.py | 14 ++++++++------ 3 files changed, 17 insertions(+), 11 deletions(-) diff --git a/appveyor.yml b/appveyor.yml index 7b413faee5c78..25a8b0542a653 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -187,16 +187,19 @@ build_script: - appveyor PushArtifact out/Default/mksnapshot.zip - appveyor PushArtifact out/Default/hunspell_dictionaries.zip - appveyor PushArtifact out/Default/electron.lib + # Temporarily disable symbol generation on 32-bit Windows due to failures - ps: >- - if ($env:GN_CONFIG -eq 'release') { + if ($env:GN_CONFIG -eq 'release' -And $env:TARGET_ARCH -ne 'ia32') { # Needed for msdia140.dll on 64-bit windows $env:Path += ";$pwd\third_party\llvm-build\Release+Asserts\bin" ninja -C out/Default electron:electron_symbols } - ps: >- if ($env:GN_CONFIG -eq 'release') { - python electron\script\zip-symbols.py - appveyor-retry appveyor PushArtifact out/Default/symbols.zip + if ($env:TARGET_ARCH -ne 'ia32') { + python electron\script\zip-symbols.py + appveyor-retry appveyor PushArtifact out/Default/symbols.zip + } } else { # It's useful to have pdb files when debugging testing builds that are # built on CI. diff --git a/script/release/release.js b/script/release/release.js index 3527f39ce2bd3..52771ba60ad91 100755 --- a/script/release/release.js +++ b/script/release/release.js @@ -131,8 +131,9 @@ function assetsForVersion (version, validatingRelease) { `electron-${version}-mas-arm64-dsym-snapshot.zip`, `electron-${version}-mas-arm64-symbols.zip`, `electron-${version}-mas-arm64.zip`, - `electron-${version}-win32-ia32-pdb.zip`, - `electron-${version}-win32-ia32-symbols.zip`, + // TODO(jkleinsc) Symbol generation on 32-bit Windows is temporarily disabled due to failures + // `electron-${version}-win32-ia32-pdb.zip`, + // `electron-${version}-win32-ia32-symbols.zip`, `electron-${version}-win32-ia32.zip`, `electron-${version}-win32-x64-pdb.zip`, `electron-${version}-win32-x64-symbols.zip`, diff --git a/script/release/uploaders/upload.py b/script/release/uploaders/upload.py index 88fbcc2423770..0c47126bcdda1 100755 --- a/script/release/uploaders/upload.py +++ b/script/release/uploaders/upload.py @@ -76,9 +76,10 @@ def main(): shutil.copy2(os.path.join(OUT_DIR, 'dist.zip'), electron_zip) upload_electron(release, electron_zip, args) if get_target_arch() != 'mips64el': - symbols_zip = os.path.join(OUT_DIR, SYMBOLS_NAME) - shutil.copy2(os.path.join(OUT_DIR, 'symbols.zip'), symbols_zip) - upload_electron(release, symbols_zip, args) + if get_target_arch() != 'ia32' and PLATFORM != 'win32': + symbols_zip = os.path.join(OUT_DIR, SYMBOLS_NAME) + shutil.copy2(os.path.join(OUT_DIR, 'symbols.zip'), symbols_zip) + upload_electron(release, symbols_zip, args) if PLATFORM == 'darwin': if get_platform_key() == 'darwin' and get_target_arch() == 'x64': api_path = os.path.join(ELECTRON_DIR, 'electron-api.json') @@ -95,9 +96,10 @@ def main(): 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) - upload_electron(release, pdb_zip, args) + if get_target_arch() != 'ia32': + pdb_zip = os.path.join(OUT_DIR, PDB_NAME) + shutil.copy2(os.path.join(OUT_DIR, 'pdb.zip'), pdb_zip) + upload_electron(release, pdb_zip, args) elif PLATFORM == 'linux': debug_zip = os.path.join(OUT_DIR, DEBUG_NAME) shutil.copy2(os.path.join(OUT_DIR, 'debug.zip'), debug_zip)