[GRASS-dev] Re: [GRASSGUI] Re: [grass-addons] r659 - trunk/grassaddons/gui/gui_modules

[Michael Barton]

On 5/17/07 6:30 PM, "Glynn Clements" <glynn at gclements.plus.com> wrote:

> 
> Michael Barton wrote:
> 
>>>> This is helpful. But I will wait to make any changes until this is worked
>>>> out--since it looks like we will also need to redefine the original command
>>>> string (i.e., to make it a proper list) prior to calling cmd.Command.
>>> 
>>> I've fixed cmd.Command itself and all of the straightforward uses.
>> 
>> So we'll need to send it a list consisting of [command+flags, option=val,
>> option=val,...]. Right?
> 
> Example:
> 
>         dbDescribe = cmd.Command (cmd = ["db.describe", "-c",
> "table=%s" % self.parent.tablename,
> "driver=%s" % self.parent.driver,
> "database=%s" % self.parent.database])
> 

Thanks. This is a good example.

Michael

> 
>>> AFAICT, GUI.parseCommand() is probably still broken:
>>> 
>>> menuform.py:1075:        cmdlst = cmd.split(' ')
>> 
>> This actually might be fine. It looks like it is only checking to see if
>> there is a space in the command string and returning an error message is
>> there is. It making sure that the only thing being passed on to menuform is
>> a simple GRASS command without arguments.
> 
> Ah; okay.

__________________________________________
Michael Barton, Professor of Anthropology
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

phone: 480-965-6213
fax: 480-965-7671
www: http://www.public.asu.edu/~cmbarton