SuperTux / supertux

SuperTux source code
https://supertux.org
GNU General Public License v3.0
2.48k stars 474 forks source link

Handle errors with a stacktrace dialog #2908

Closed MatusGuy closed 2 months ago

tobbi commented 2 months ago

Can something like this also exist for Linux and MacOS?

MatusGuy commented 2 months ago

like, a dialog? Not sure about MacOS, but on Linux there is no universal way to create a dialog, and I wouldn't want to rely on xmessage

tobbi commented 2 months ago

Couldn't we have an error handler application that posts stacktraces to some REST API endpoint?

MatusGuy commented 2 months ago

you mean, post stacktraces to a server??? just make a github issue...

tobbi commented 2 months ago

I meant collect stacktraces on our server the way mozilla does.

tobbi commented 2 months ago

We should switch to RelWithDebInfo builds for both nightly and release versions. That way we can get line numbers from stacktraces.

mrkubax10 commented 2 months ago

Can something like this also exist for Linux and MacOS?

We already print stacktrace in terminal when running on Linux with glibc.

tobbi commented 2 months ago

Do we include the main supertux config file contents and / or graphics driver here?

tobbi commented 2 months ago

A couple more things:

tobbi commented 2 months ago

We're having a problem:

The call to exit() in the error handler tries to call a bunch of stuff (in "atexit") and therefore SuperTux lingers indefinitely. Any idea what to d?

tobbi commented 2 months ago

Okay, so this fixes my issues with the program not exiting.