[GRASS-dev] [GRASS GIS] #630: "Mapset Access" broken (wxpython)

[Martin Landa]

Hi,

2009/6/14 Cline, Royce L. <rcline at nd.gov>:

[...]

> In lines 1800-1805 of preferences.py the call to g.mapsets -l is made on
> line1800.
>        ret = gcmd.RunCommand('g.mapsets',
>                              flags = 'l',
>                              read = True)
>        mapsets = []
>        if ret:
>            mapsets = ret.rstrip(' \n').split(' ')
>
> In line 1805, ret.rstrip(' \n').split(' '), only the trailing line feed is
> removed and not the line feed following the first line. Line 1805 should be
> replaced with:
> mapsets = ret.replace('\n','').split()
>
> That way both the last line feed and any line feeds on multiple lines are
> removed. Using .split() removes the need to handle the trailing space.
>
> The change fixes the problem for me in 6.4RC4 and 6.4RC5. It also fixed a
> crash when starting wxpython and opening an existing workspace. Can someone
> with svn access verify my solution and fix this.

done in r37880.

Regards, Martin

-- 
Martin Landa <landa.martin gmail.com> * http://gama.fsv.cvut.cz/~landa