Opened 14 years ago

Last modified 8 years ago

#457 new defect

[PATCH] Default crashlog location is bad

Reported by: Philip Taylor Owned by:
Priority: If Time Permits Milestone: Backlog
Component: Core engine Keywords: patch
Cc: Jan Wassenberg Patch:

Description

By default (before it has called psSetLogDir), the game tries to write crashlog.txt into the current directory, which is typically the system directory. When the game is installed in a read-only directory, and an error occurs during the startup process, that causes obvious problems. The original error report is lost and replaced by the can't-write-crashlog.txt error report, which isn't very helpful.

On Linux, maybe it'd be best to detect any errors writing crashlog.txt and then just dump the original error to stdout instead, because people are likely to see it there. On Windows maybe it should ignore crashlog-writing-errors and just display the normal dialog box for the original error. Or something else.

Attachments (2)

t457.patch (3.2 KB ) - added by Sergey Kushnirenko 8 years ago.
t457.2.patch (3.2 KB ) - added by Sergey Kushnirenko 8 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 by (none), 14 years ago

Milestone: Unclassified

Milestone Unclassified deleted

comment:2 by Andrew, 14 years ago

Milestone: OS Alpha 2
Owner: set to Andrew

comment:3 by Kieran P, 14 years ago

Milestone: OS Alpha 2OS Alpha 3

comment:4 by Kieran P, 14 years ago

Priority: minortrivial

comment:5 by Kieran P, 14 years ago

Owner: Andrew removed

comment:6 by Kieran P, 14 years ago

Milestone: Alpha 3Alpha 4

comment:7 by Kieran P, 13 years ago

Milestone: Alpha 4Alpha 5

comment:8 by Kieran P, 13 years ago

Milestone: Alpha 5Backlog

by Sergey Kushnirenko, 8 years ago

Attachment: t457.patch added

comment:9 by Sergey Kushnirenko, 8 years ago

Keywords: review patch added
Milestone: BacklogAlpha 20
Summary: Default crashlog location is bad[PATCH] Default crashlog location is bad

by Sergey Kushnirenko, 8 years ago

Attachment: t457.2.patch added

comment:10 by Itms, 8 years ago

Milestone: Alpha 20Alpha 21

comment:11 by Itms, 8 years ago

Keywords: review removed

This patch does not implement what is proposed in the ticket, which is a proper handling of the read-only situation.

I don't know if using /var/log is a perfect solution and the Windows case stays uncovered.

comment:12 by elexis, 8 years ago

Backlogging due to lack of progress.

comment:13 by elexis, 8 years ago

Milestone: Alpha 21Backlog
Note: See TracTickets for help on using tickets.