[GRASS-dev] Re: [GRASS GIS] #1507: r.in.gdal & others: -e must only modify the current mapset's WIND file

[GRASS GIS]

#1507: r.in.gdal & others: -e must only modify the current mapset's WIND file
---------------------------------------------+------------------------------
 Reporter:  hamish                           |       Owner:  grass-dev@…              
     Type:  defect                           |      Status:  new                      
 Priority:  blocker                          |   Milestone:  6.4.2                    
Component:  Default                          |     Version:  svn-trunk                
 Keywords:  r.in.gdal, r.external, v.in.ogr  |    Platform:  All                      
      Cpu:  All                              |  
---------------------------------------------+------------------------------

Comment(by glynn):

 Replying to [comment:3 hamish]:

 > backwards compatibility is allowed to be broken in cases where the
 previous behaviour was clearly in error and harming the user.

 Which doesn't apply here, IMHO.

 Updating the default region rather than the current region was clearly
 intentional. The default window provides information about the data in the
 location. The current window is an operational parameter.

 I'm not sure that having a large number of r.in.* modules, each having the
 ability to update the default region is the best approach. Having the
 modules update the current region then using an extra "g.region -s" step
 may be preferable.

 OTOH, it's not unreasonable to ignore the locking issue when changing the
 default window, as the default window shouldn't be used for much beyond
 "g.region -d". Also, it should be sufficient to have write permission for
 the DEFAULT_WIND file; it shouldn't be necessary to own the containing
 mapset (that requirement exists mainly to deal with issues relating to
 subdirectories, which don't apply here).

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