Opened 11 months ago

Closed 7 months ago

#14777 closed enhancement (fixed)

[debug_server] move Save core file to Debugger

Reported by: diver Owned by: waddlesplash
Priority: normal Milestone: Unscheduled
Component: Servers/debug_server Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

Ever since the introduction of core file support it was very rarely used to debug things (almost never judging from Trac attachments). Yet saving core file option is visible to user every the time some app crashes. Additionally, in certain locales crash dialog could be pretty wide.

An alternative would be to move this functionality to Debugger menu.

Change History (4)

comment:1 by diver, 11 months ago

Type: bugenhancement

comment:2 by pulkomandy, 11 months ago

It is possibly useful in cases where Debugger is not available, not working or not convenient. For example it is not installed, or it crashes when trying to debug the app.

I agree it may not be something to put directly in every users hands, but users should never see that crash dialog anyway, right? ;) So we can be a little more technical than usual here.

We can review the layout of that window if it's too wide. Maybe put the buttons vertically instead of horizontally?

comment:3 by waddlesplash, 7 months ago

Debugger crashing, but "save report" / "write core file" not crashing, is an exceptional case that occurs very rarely, if ever, I think? So we don't need to worry about that.

Debugger is also included in the base system image, so it's always installed, no concern here.

The window is a standard info window, we don't want to make it look strange for no real reason.

So, moving it into the Debugger menu makes the most sense here. There's already a CLI command for it, adding a menu entry should be pretty easy.

comment:4 by waddlesplash, 7 months ago

Resolution: fixed
Status: assignedclosed

Done in hrev53104.

Note: See TracTickets for help on using tickets.