Opened 14 years ago

Closed 14 years ago

#194 closed defect (fixed)

i.fusion.brovey: remove temp region file?

Reported by: hamish Owned by: grass-dev@…
Priority: minor Milestone: 6.4.0
Component: Default Version: svn-trunk
Keywords: i.fusion.brovey Cc:
CPU: Unspecified Platform: Unspecified



just looking through the i.fusion.brovey script I see that the cleanup() section resets the region to the temporary region file instead removing it. Which, AFAICT, is a noop and leaves a saved region file behind.

It in not important that the WIND_OVERRIDE variable is not unset there -- that goes away automatically as soon as the script exits.



Change History (4)

comment:1 by hamish, 14 years ago

Also, it is unclear to me if the "clone current region" step is really needed before setting WIND_OVERRIDE and using the new region file. I notice doesn't bother to do that.

ie can WIND_OVERRIDE legally point to a region wind file which does not yet exist?


comment:2 by neteler, 14 years ago

AFAIR WIND_OVERRIDE didn't exist when I was writing i.fusion.brovey. Feel free to make appropriate fixes.


in reply to:  1 comment:3 by glynn, 14 years ago

Replying to hamish:

ie can WIND_OVERRIDE legally point to a region wind file which does not yet exist?

It depends upon the situation, but in most cases, no. Anything which tries to read the window (which lots of functions do implicitly) will fail if WIND_OVERRIDE references a non-existent region.

In particular, g.region calls G_get_window() unless -d is given (any command line settings override specific components of the current region), which will fail if WIND_OVERRIDE is set but the region doesn't exit.

comment:4 by hamish, 14 years ago

Resolution: fixed
Status: newclosed

fixed in r31829, r31830, r31832.


Note: See TracTickets for help on using tickets.