r.in.srtm.region add-on updates for 7.1svn

[awickert]

The simplification in GRASS 7.1 that changes g.list/g.mlist and g.mremove into g.list and g.remove for everything broke some of the functionality of r.in.srtm.region. I've attached a revised file and a list of diffs. My changes check for GRASS version, and while I haven't tested it with 7.0, should allow it to work on both 7.0 and 7.1.

I should note that these changes likely broke other add-ons, but that I am in support of the g.list/g.remove change.

Thanks in advance for updating,

Andy Wickert

[awickert]

Updated r.in.srtm.region

[awickert]

diffs

[mmetz]

Replying to awickert:

The simplification in GRASS 7.1 that changes g.list/g.mlist and g.mremove into g.list and g.remove for everything broke some of the functionality of r.in.srtm.region. I've attached a revised file and a list of diffs. My changes check for GRASS version, and while I haven't tested it with 7.0, should allow it to work on both 7.0 and 7.1.

Thanks for the patch. I have updated r.in.srtm.region in r62451. Note that the default is now to import SRTM v3.

[awickert]

Thanks, Markus! Looks good to me, so I am marking this as "resolved".

[neteler]

Milestone renamed