[GRASS-dev] [GRASS GIS] #54: g.remove mixes raster and vector maps: "Remove reclassed map first"

[Markus Neteler]

Glynn

the suggested patch works. Please submit...

thanks
Markus

On Wed, Feb 20, 2008 at 4:48 AM, Glynn Clements
<glynn at gclements.plus.com> wrote:
>
>
>  GRASS GIS wrote:
>
>  > #54: g.remove mixes raster and vector maps: "Remove reclassed map first"
>  > ---------------------+------------------------------------------------------
>  >  Reporter:  neteler  |       Owner:  grass-dev at lists.osgeo.org
>  >      Type:  defect   |      Status:  new
>  >  Priority:  major    |   Milestone:  6.4.0
>  > Component:  default  |     Version:  unspecified
>  >  Keywords:           |
>  > ---------------------+------------------------------------------------------
>  >  A strange bug appeared: Trying to remove a vector map, g.remove gets
>  >  confused with a non-exiting reclass map
>  >
>  >  {{{
>  >  GRASS 6.3.svn (pat):~ > g.remove
>  >  vect=lst_20050101_avg,lst_20050101_avg2,lst_200501_avg_filt
>  >  WARNING: [lst_20050101_avg at modisLSTzanzara] is a base map. Remove
>  >  reclassed
>  >           map first: MASK at modisLSTzanzara
>  >  Removing vector <lst_20050101_avg2>
>  >  WARNING: [lst_200501_avg_filt at modisLSTzanzara] is a base map. Remove
>  >           reclassed map first: MASK at modisLSTzanzara
>  >
>  >  GRASS 6.3.svn (pat):~ > g.remove vect=lst_20050101_avg
>  >  WARNING: [lst_20050101_avg at modisLSTzanzara] is a base map. Remove
>  >  reclassed
>  >           map first: MASK at modisLSTzanzara
>  >
>  >  GRASS 6.3.svn (pat):~ > g.list rast | grep lst_20050101_avg
>  >  lst_20030114_avg                        lst_20050101_avg
>  >  lst_20030115_avg                        lst_20050101_avg_lstmodel
>  >
>  >  GRASS 6.3.svn (pat):~ > g.list vect | grep lst_
>  >  lst_20050101_avg    lst_200501_avg_filt
>  >
>  >  GRASS 6.3.svn (pat):~ > r.info lst_20050101_avg | grep generated
>  >   |    generated by r.series
>  >  |
>  >
>  >  GRASS 6.3.svn (pat):~ > g.gisenv
>  >  GISDBASE=/home/neteler/grassdata
>  >  LOCATION_NAME=pat
>  >  MAPSET=modisLSTzanzara
>  >  DEBUG=0
>  >  GRASS_GUI=text
>  >  MONITOR=x0
>  >  }}}
>  >
>  >  The problem seems to be in source:grass/trunk/general/manage/cmd/remove.c
>  >  (line 83) where it should not enter. Fix appreciated..
>
>  Can you test the attached patch?
>
>  --
>  Glynn Clements <glynn at gclements.plus.com>