Skip to content

Crash reporting and RelWithDebInfo - #2366

Draft
JulianGro wants to merge 13 commits into
overte-org:masterfrom
JulianGro:RelWithDebInfo
Draft

Crash reporting and RelWithDebInfo#2366
JulianGro wants to merge 13 commits into
overte-org:masterfrom
JulianGro:RelWithDebInfo

Conversation

@JulianGro

@JulianGro JulianGro commented Jul 15, 2026

Copy link
Copy Markdown
Member

Idea behind this PR is to bring back crash reporting, but in a way that yields actually useful crash reports.
In the past, we would collect crash reports, but without debug information and with the builds fully optimized. This would essentially only tell us that someone was crashing.
Now I want to create RelWithDebInfo builds, automatically upload the debug information to Sentry, and strip the debug symbols in the later packaging step.

Conan recipe change: overte-org/overte-conan-recipes#59

I tested the Sentry uploading locally using act, since Pull Requests cannot usually access secrets.

Known issues:

  • The "Interface Crashed Last Run" dialogue will say that "Unfortunately, crash reporting isn't built into this release." because it checks if the crashpad_handler is running before it was started.
    Maybe this was broken during the Application.cpp refactor?
    Crashpad_handler starts successfully afterward.
    Crash reporting can still be enabled in the general settings.
    Enabling crash reporting in the settings somehow fixes the issue as well.
  • Logs aren't attached to the crash reports anymore. This may or may not be related to the following error message:
ERROR directory_reader_posix.cc:43] opendir /home/juliangro/.local/share/Overte - Dev/Interface/crashpad-db/attachments/886770c6-7a69-400e-97e5-9aad8311b3c5: No such file or directory (2)
  • Most additional information appears to be missing, such as avatar URL, visited Domain server and graphics driver vendor.
  • Crashpad doesn't build on MSVC anymore, so we only enable it on Linux and macOS for now.

Todo:

  • Use Sentry-native instead of Crashpad.
  • OVERTE_USE_SENTRY

@JulianGro
JulianGro force-pushed the RelWithDebInfo branch 2 times, most recently from 4fecacf to 542027d Compare July 17, 2026 19:29
@JulianGro JulianGro added needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested labels Jul 17, 2026
@JulianGro
JulianGro marked this pull request as ready for review July 17, 2026 20:08
@JulianGro
JulianGro marked this pull request as draft July 22, 2026 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant