[GRASS-dev] [GRASS-SVN] r62566 - in grass/trunk: gui/wxpython/vdigit lib/python/exceptions lib/python/script lib/python/temporal scripts/d.shadedmap scripts/db.dropcolumn scripts/db.in.ogr scripts/db.out.ogr scripts/db.test scripts/g.extension.all scr

On Mon, Nov 17, 2014 at 10:03 AM, Markus Neteler <neteler@osgeo.org> wrote:

Do you plan to backport this (and related) fixes?

Yes. I guess I should do it ASAP. With Anna we have already prepared patch

for addons. I was waiting for some problems such as the one with
grass.script.core.message() to appear. Now it seems that it is settled
well. I have used few functions in GUI recently too, so most of the code
should be OK now. There are few issues here and there but it should be only
replacing one error message by another. I will try to do it today or
tomorrow.

However, there are also the basename/multiple inputs/outputs related
changes which are not done yet and should go to 7.0. I'm uncomfortable with
doing this last minute change but the improvements would be significant.

Vaclav

On Mon, Nov 3, 2014 at 6:32 AM, <svn_grass@osgeo.org> wrote:

> Author: wenzeslaus
> Date: 2014-11-02 21:32:32 -0800 (Sun, 02 Nov 2014)
> New Revision: 62566
>
> Modified:
> grass/trunk/gui/wxpython/vdigit/g.gui.vdigit.py
> grass/trunk/lib/python/exceptions/__init__.py
> grass/trunk/lib/python/script/array.py
> grass/trunk/lib/python/script/core.py
> grass/trunk/lib/python/script/db.py
> grass/trunk/lib/python/script/raster.py
> grass/trunk/lib/python/script/raster3d.py
> grass/trunk/lib/python/temporal/aggregation.py
> grass/trunk/lib/python/temporal/extract.py
> grass/trunk/lib/python/temporal/mapcalc.py
> grass/trunk/lib/python/temporal/stds_export.py
> grass/trunk/lib/python/temporal/stds_import.py
> grass/trunk/scripts/d.shadedmap/d.shadedmap.py
> grass/trunk/scripts/db.dropcolumn/db.dropcolumn.py
> grass/trunk/scripts/db.in.ogr/db.in.ogr.py
> grass/trunk/scripts/db.out.ogr/db.out.ogr.py
> grass/trunk/scripts/db.test/db.test.py
> grass/trunk/scripts/g.extension.all/g.extension.all.py
> grass/trunk/scripts/i.in.spotvgt/i.in.spotvgt.py
> grass/trunk/scripts/r.fillnulls/r.fillnulls.py
> grass/trunk/scripts/r.grow/r.grow.py
> grass/trunk/scripts/r.in.srtm/r.in.srtm.py
> grass/trunk/scripts/r.in.wms/wms_base.py
> grass/trunk/scripts/r.mask/r.mask.py
> grass/trunk/scripts/r.out.xyz/r.out.xyz.py
> grass/trunk/scripts/r3.in.xyz/r3.in.xyz.py
> grass/trunk/scripts/v.build.all/v.build.all.py
> grass/trunk/scripts/v.convert.all/v.convert.all.py
> grass/trunk/scripts/v.db.addtable/v.db.addtable.py
> grass/trunk/scripts/v.db.dropcolumn/v.db.dropcolumn.py
> grass/trunk/scripts/v.db.droprow/v.db.droprow.py
> grass/trunk/scripts/v.db.droptable/v.db.droptable.py
> grass/trunk/scripts/v.db.join/v.db.join.py
> grass/trunk/scripts/v.db.reconnect.all/v.db.reconnect.all.py
> grass/trunk/scripts/v.dissolve/v.dissolve.py
> grass/trunk/scripts/v.in.e00/v.in.e00.py
> grass/trunk/scripts/v.in.gps/v.in.gps.py
> grass/trunk/scripts/v.in.mapgen/v.in.mapgen.py
> grass/trunk/scripts/v.in.wfs/v.in.wfs.py
> grass/trunk/scripts/v.out.gps/v.out.gps.py
> grass/trunk/scripts/v.rast.stats/v.rast.stats.py
> grass/trunk/scripts/v.to.lines/v.to.lines.py
> grass/trunk/scripts/v.unpack/v.unpack.py
> grass/trunk/scripts/v.what.vect/v.what.vect.py
> grass/trunk/temporal/t.rast.colors/t.rast.colors.py
> grass/trunk/temporal/t.rast.out.vtk/t.rast.out.vtk.py
> grass/trunk/temporal/t.rast.series/t.rast.series.py
> grass/trunk/temporal/t.rast.to.rast3/t.rast.to.rast3.py
> grass/trunk/temporal/t.vect.observe.strds/t.vect.observe.strds.py
> grass/trunk/temporal/t.vect.what.strds/t.vect.what.strds.py
> Log:
> pythonlib: raise exceptions from run_command and others (#2326)
>
> * run_command and write_command are raising exception instead of
returning subprocess retrun code
> * read_command is raising instead of being silent in case of error
(same for parse_command)
> * code using/checking return code changed to try-except, rest left as
it was
> * basic functions in GUI and tests are running
>
....

On Mon, Nov 17, 2014 at 4:28 PM, Vaclav Petras <wenzeslaus@gmail.com> wrote:
...

However, there are also the basename/multiple inputs/outputs related changes
which are not done yet and should go to 7.0. I'm uncomfortable with doing
this last minute change but the improvements would be significant.

We are not in freeze but preparing beta4, so no problem.
Better getting improvements in rather than postpone them to GRASS GIS 8...

My 0.02 cents,
Markus