On Tue, Dec 1, 2015 at 7:24 PM, Vaclav Petras <[email protected]> wrote: > Hi Helmut, > > On Tue, Dec 1, 2015 at 5:11 AM, <[email protected]> wrote: >> >> +try: >> + import pyspatialite.dbapi2 as db >> +except ImportError: >> + grass.warning( "pyspatialite has to be installed." ) > > > better than this is the approach taken in v.flexture where you let parser > run and then you import the special dependencies. v.class.mlpy and > especially v.class.ml are more complicated cases where the dependencies are > spread more through the code but similar approach is still applicable. > > As you can see from v.flexture, there was an idea to provide a way how to > install the dependencies but this might be too difficult and suggestions in > the error message or manual might be just the right thing to do. > > Vaclav > > https://trac.osgeo.org/grass/browser/grass-addons/grass7/vector/v.flexure/v.flexure.py > https://trac.osgeo.org/grass/browser/grass-addons/grass7/vector/v.class.mlpy/v.class.mlpy.py > https://trac.osgeo.org/grass/browser/grass-addons/grass7/vector/v.class.ml/v.class.ml.py
FWIW, today I have added a termcolor check to g.search.modules, just within the code. https://trac.osgeo.org/grass/changeset/67001 Only if the user requested -c (colorize) the check is executed (hence, after the parser was called). Hope this makes sense, Markus _______________________________________________ grass-dev mailing list [email protected] http://lists.osgeo.org/mailman/listinfo/grass-dev
