[GRASS-dev] [GRASS GIS] #1407: confusing error message after grass7 crash

GDAL · July 21, 2011, 7:55pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: | Platform: All
Cpu: Unspecified |
-------------------------+--------------------------------------------------
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.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 21, 2011, 7:59pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: | Platform: All
Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by MilenaN):

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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:1>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 21, 2011, 8:00pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: wingrass | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by martinl):

* keywords: => wingrass
* platform: All => Unspecified

Comment:

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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:2>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 22, 2011, 11:56am

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by neteler):

* keywords: wingrass =>

Comment:

The problem is similar on Linux, see attachment.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:3>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 22, 2011, 6:55pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by martinl):

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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:4>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 22, 2011, 6:56pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by martinl):

Replying to [comment:1 MilenaN]:
> Checking with the file expolorer, the indicated file does not exist.

this should only refer to wingrass...

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:5>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 22, 2011, 6:57pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: martinl
Type: enhancement | Status: assigned
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by martinl):

* cc: grass-dev@… (added)
* owner: grass-dev@… => martinl
* status: new => assigned

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:6>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 22, 2011, 6:57pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: martinl
Type: enhancement | Status: assigned
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: wxgui | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by martinl):

* keywords: => wxgui

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:7>
GRASS GIS <http://grass.osgeo.org>

GDAL · July 26, 2011, 2:47pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: martinl
Type: enhancement | Status: assigned
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: wxgui | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by martinl):

Replying to [comment:4 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.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:8>
GRASS GIS <http://grass.osgeo.org>

GDAL · August 1, 2011, 6:03pm

#1407: confusing error message after grass7 crash
-------------------------+--------------------------------------------------
Reporter: MilenaN | Owner: martinl
Type: enhancement | Status: assigned
Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Keywords: wxgui | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by martinl):

Replying to [comment:8 martinl]:
> Replying to [comment:4 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}}}?

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:9>
GRASS GIS <http://grass.osgeo.org>

GDAL · September 12, 2011, 10:30pm

#1407: confusing error message after grass7 crash
--------------------------+-------------------------------------------------
  Reporter: MilenaN | Owner: martinl
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 6.4.2
Component: Startup | Version: svn-trunk
Resolution: fixed | Keywords: wxgui
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by martinl):

* status: assigned => closed
* resolution: => fixed

Comment:

Replying to [comment:9 martinl]:
> Replying to [comment:8 martinl]:
> > Replying to [comment:4 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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:10>
GRASS GIS <http://grass.osgeo.org>

GDAL · September 12, 2011, 10:30pm

#1407: confusing error message after grass7 crash
--------------------------+-------------------------------------------------
  Reporter: MilenaN | Owner: martinl
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 6.4.2
Component: wxGUI | Version: svn-trunk
Resolution: fixed | Keywords: startup
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by martinl):

* keywords: wxgui => startup
* component: Startup => wxGUI

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:11>
GRASS GIS <http://grass.osgeo.org>

GDAL · September 12, 2011, 11:19pm

#1407: confusing error message after grass7 crash
--------------------------+-------------------------------------------------
  Reporter: MilenaN | Owner: martinl
      Type: enhancement | Status: reopened
  Priority: normal | Milestone: 6.4.2
Component: wxGUI | Version: svn-trunk
Resolution: | Keywords: startup
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by hamish):

* status: closed => reopened
* resolution: fixed =>

Comment:

Replying to [comment:4 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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:12>
GRASS GIS <http://grass.osgeo.org>

GDAL · September 30, 2011, 2:54pm

#1407: confusing error message after grass7 crash
--------------------------+-------------------------------------------------
  Reporter: MilenaN | Owner: martinl
      Type: enhancement | Status: reopened
  Priority: blocker | Milestone: 6.4.2
Component: wxGUI | Version: svn-trunk
Resolution: | Keywords: startup
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by martinl):

* priority: normal => blocker

Comment:

Replying to [comment:12 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?

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:13>
GRASS GIS <http://grass.osgeo.org>

GDAL · October 30, 2011, 11:05pm

#1407: confusing error message after grass7 crash
--------------------------+-------------------------------------------------
  Reporter: MilenaN | Owner: martinl
      Type: enhancement | Status: closed
  Priority: blocker | Milestone: 6.4.2
Component: wxGUI | Version: svn-trunk
Resolution: fixed | Keywords: startup
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by martinl):

* status: reopened => closed
* resolution: => fixed

Comment:

Replying to [comment:13 martinl]:

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

Backported to relbr64 in r49016

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1407#comment:14>
GRASS GIS <http://grass.osgeo.org>