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)
Change History (15)
comment:1 by , 14 years ago
Milestone: | Unclassified |
---|
comment:2 by , 14 years ago
Milestone: | → OS Alpha 2 |
---|---|
Owner: | set to |
comment:3 by , 14 years ago
Milestone: | OS Alpha 2 → OS Alpha 3 |
---|
comment:4 by , 14 years ago
Priority: | minor → trivial |
---|
comment:5 by , 14 years ago
Owner: | removed |
---|
comment:6 by , 14 years ago
Milestone: | Alpha 3 → Alpha 4 |
---|
comment:7 by , 13 years ago
Milestone: | Alpha 4 → Alpha 5 |
---|
comment:8 by , 13 years ago
Milestone: | Alpha 5 → Backlog |
---|
by , 8 years ago
Attachment: | t457.patch added |
---|
comment:9 by , 8 years ago
Keywords: | review patch added |
---|---|
Milestone: | Backlog → Alpha 20 |
Summary: | Default crashlog location is bad → [PATCH] Default crashlog location is bad |
by , 8 years ago
Attachment: | t457.2.patch added |
---|
comment:10 by , 8 years ago
Milestone: | Alpha 20 → Alpha 21 |
---|
comment:11 by , 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:13 by , 8 years ago
Milestone: | Alpha 21 → Backlog |
---|
Milestone Unclassified deleted