Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Symbol generation failing on 32-bit Windows release builds (nightlies) #34413

Closed
3 tasks done
VerteDinde opened this issue Jun 1, 2022 · 2 comments
Closed
3 tasks done
Assignees
Labels
bug 🪲 main upgrade-follow-up Stuff left to do after a Chromium upgrade.

Comments

@VerteDinde
Copy link
Member

Preflight Checklist

Electron Version

main

What operating system are you using?

Windows

Operating System Version

Windows

What arch are you using?

ia32

Last Known Working Electron version

20.0.0

Expected Behavior

Release builds succeed

Actual Behavior

Symbol generation on 32-bit Windows release builds on nightlies and 19-x-y are failing with the following error:

FAILED: gen/electron/electron_app_symbols.stamp 
C:/depot_tools/bootstrap-2@3_8_10_chromium_23_bin/python3/bin/python3.exe ../../electron/build/dump_syms.py ./dump_syms.exe electron.exe breakpad_symbols gen/electron/electron_app_symbols.stamp
failed to get function name
WriteSymbols failed.
Traceback (most recent call last):
  File "../../electron/build/dump_syms.py", line 53, in <module>
    main(*sys.argv[1:])
  File "../../electron/build/dump_syms.py", line 42, in main
    symbol_data = subprocess.check_output(args).decode(sys.stdout.encoding)
  File "C:\depot_tools\bootstrap-2@3_8_10_chromium_23_bin\python3\bin\lib\subprocess.py", line 415, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "C:\depot_tools\bootstrap-2@3_8_10_chromium_23_bin\python3\bin\lib\subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['./dump_syms.exe', 'electron.exe']' returned non-zero exit status 1.
ninja: build stopped: subcommand failed.
Command exited with code 1

Testcase Gist URL

https://ci.appveyor.com/project/electron-bot/electron-ia32-release/build/1.0.1818

Additional Information

Introduced in #34272

@VerteDinde VerteDinde added bug 🪲 main upgrade-follow-up Stuff left to do after a Chromium upgrade. labels Jun 1, 2022
@VerteDinde VerteDinde self-assigned this Jun 1, 2022
@khwaaj
Copy link
Contributor

khwaaj commented Jun 2, 2022

I ran into this recently when building the v20 alpha for win-ia32. Maybe you are already aware, but the cause is that dump_syms.exe actually runs out of available virtual address space for 32-bit executables on Windows.

Generating the symbols does work if you use the win-x64 version of dump_syms.exe instead, so one solution would be to always build the host tooling as 64-bit.

Just a FYI that could maybe save you some time if you were not already aware of it 😄

@RaisinTen
Copy link
Member

Closing since #35078 has landed on main, thanks for the fix. Feel free to reopen if you want this to stay open till the fix gets backported to 18-x-y since that branch was affected too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 main upgrade-follow-up Stuff left to do after a Chromium upgrade.
Projects
No open projects
Status: Fixed for Next Release
Development

No branches or pull requests

3 participants