Opened 11 years ago

Closed 11 years ago

#1407 closed enhancement (fixed)

confusing error message after grass7 crash

Reported by: MilenaN Owned by: martinl
Priority: blocker Milestone: 6.4.2
Component: wxGUI Version: svn-trunk
Keywords: startup Cc: grass-dev@…
CPU: Unspecified Platform: Unspecified

Description

After crashed grass7 session grass complains about .gislock file. The message is unhelpful if the user doesn't know what to do. Please add an explanation for the needed step to fix the blocked grass installation.

Attachments (3)

leftover_gislock.png (32.0 KB ) - added by MilenaN 11 years ago.
.gislock file present but no idea how to continue
linux_gislock.png (14.2 KB ) - added by neteler 11 years ago.
Crashed GRASS session on Linux
wxgui_gislock.png (16.6 KB ) - added by martinl 11 years ago.
remove gislock?

Download all attachments as: .zip

Change History (17)

by MilenaN, 11 years ago

Attachment: leftover_gislock.png added

.gislock file present but no idea how to continue

comment:1 by MilenaN, 11 years ago

Checking with the file expolorer, the indicated file does not exist.

comment:2 by martinl, 11 years ago

Keywords: wingrass added
Platform: AllUnspecified

I think it's related only to Windows...

by neteler, 11 years ago

Attachment: linux_gislock.png added

Crashed GRASS session on Linux

comment:3 by neteler, 11 years ago

Keywords: wingrass removed

The problem is similar on Linux, see attachment.

by martinl, 11 years ago

Attachment: wxgui_gislock.png added

remove gislock?

comment:4 by martinl, 11 years ago

I have modified the dialog to allow remove gislock and continue (r47226) - see the attached screenshot. Would it help?

in reply to:  1 comment:5 by martinl, 11 years ago

Replying to MilenaN:

Checking with the file expolorer, the indicated file does not exist.

this should only refer to wingrass...

comment:6 by martinl, 11 years ago

Cc: grass-dev@… added
Owner: changed from grass-dev@… to martinl
Status: newassigned

comment:7 by martinl, 11 years ago

Keywords: wxgui added

in reply to:  4 ; comment:8 by martinl, 11 years ago

Replying to martinl:

I have modified the dialog to allow remove gislock and continue (r47226) - see the attached screenshot. Would it help?

Backported to devbr6 in r47246. Testing welcomed.

in reply to:  8 ; comment:9 by martinl, 11 years ago

Replying to martinl:

Replying to martinl:

I have modified the dialog to allow remove gislock and continue (r47226) - see the attached screenshot. Would it help?

Backported to devbr6 in r47246. Testing welcomed.

Before closing the ticket, should it be also backported to relbr64?

in reply to:  9 comment:10 by martinl, 11 years ago

Resolution: fixed
Status: assignedclosed

Replying to martinl:

Replying to martinl:

Replying to martinl:

I have modified the dialog to allow remove gislock and continue (r47226) - see the attached screenshot. Would it help?

Backported to devbr6 in r47246. Testing welcomed.

Before closing the ticket, should it be also backported to relbr64?

Backported as r48257

comment:11 by martinl, 11 years ago

Component: StartupwxGUI
Keywords: startup added; wxgui removed

in reply to:  4 ; comment:12 by hamish, 11 years ago

Resolution: fixed
Status: closedreopened

Replying to martinl:

I have modified the dialog to allow remove gislock and continue (r47226) -

I would argue to make the warning a lot more dire and the functionality much harder to access.

the user is blocked from starting twice for a good reason, and it's for their own safety whether they realize it or not. Too many people are used to broken software and just click past "an error occurred -> ignore" without thinking about it.

since grass can crash too, or the power go out, etc, sure it's nice to allow recovery without filesystem hacks, but perhaps have it launch a second dialog saying "Are you sure? If you really are running another GRASS session doing this could corrupt your data. Have another look in the processor manager just to be sure.." or something like that.

Hamsih

in reply to:  12 ; comment:13 by martinl, 11 years ago

Priority: normalblocker

Replying to hamish:

the user is blocked from starting twice for a good reason, and it's for their own safety whether they realize it or not. Too many people are used to broken software and just click past "an error occurred -> ignore" without thinking about it.

since grass can crash too, or the power go out, etc, sure it's nice to allow recovery without filesystem hacks, but perhaps have it launch a second dialog saying "Are you sure? If you really are running another GRASS session doing this could corrupt your data. Have another look in the processor manager just to be sure.." or something like that.

Try r48565, backported to devbr in r48566 for testing. Is it better?

in reply to:  13 comment:14 by martinl, 11 years ago

Resolution: fixed
Status: reopenedclosed

Replying to martinl:

Try r48565, backported to devbr in r48566 for testing. Is it better?

Backported to relbr64 in r49016

Note: See TracTickets for help on using tickets.