[GRASS-dev] g.list -a

Martin Landa landa.martin at gmail.com
Wed Dec 27 05:12:37 EST 2006


Hi,

2006/12/27, Glynn Clements <glynn at gclements.plus.com>:
>
> Martin Landa wrote:
>
> > > > I am trying to implement the wish [1]. I found a strange construction
> > > > in g.list module. The -f flag (full listing) calls the external
> > > > program $GISBASE/etc/lister/[cell|vector]. These little programs are
> > > > called only in g.list -f (I think).
> > > >
> > > > I rewrote g.list without calling any external program -- i.e. I moved
> > > > functions from lister directory to lib/lister.c. If there are no
> > > > serious objections I will commit it to CVS.
> > >
> > > Don't remove the existing functionality; add a new switch for using
> > > your built-in lister.
> >
> > this patch does not remove any functionality (AFAIK). The only change
> > is: the functionality of lister (programs) is moved to lib/lister.c. I
> > don't see any reason why to have this functionality (the -f flag)
> > outside of the g.list (/etc/lister/[cell|vector]). Maybe there is a
> > particular reason for this construction, please let me know. Thanks!
>

> The user can install lister programs for other types and/or replace
> existing ones.

OK, it makes sense to me (only it's hard to imagine an user how
installs other lister program instead of changing the code;-)

> The point of -f isn't so much to provide specific additional
> functionality, but to allow the handling to be deferred to an external
> program.

You are right, I have tried to change the patch in this way, not sure
if I am working with child process as I should have. In any case
g.list works:-)

Martin

> --
> Glynn Clements <glynn at gclements.plus.com>
>


-- 
Martin Landa <landa.martin at gmail.com> * http://gama.fsv.cvut.cz/~landa *
-------------- next part --------------
A non-text attachment was scrubbed...
Name: g_list_a-2.diff.gz
Type: application/x-gzip
Size: 2536 bytes
Desc: not available
Url : http://lists.osgeo.org/pipermail/grass-dev/attachments/20061227/85be7023/g_list_a-2.diff.gz


More information about the grass-dev mailing list