Paul McNett wrote:
>  > replace blokc of generated code wiht call to app.startupForms()
> 
> Hey Carl, I understand what you are trying to do (get rid of generated 
> blocks of code), but because datanav/AppWizard is a specialized use of 
> Dabo, your changes would be better not in dabo.dApp but in a new 
> datanav.App class. Having a function in base Dabo that expects to find 
> classes starting with "Frm" in a ui/ directory isn't right, since it was 
> AppWizard that created that structure, not Dabo itself.

Yeah, I was wondering about the "Frm" stuff hardcoded in there.  I like the 
idea 
of being able to do "that" from the command line, but not sure if this is time 
to break backwards compatibility.  I am working on some other command line 
things that may be "better" and still be backwards compatable.  stay tuned.

> 
> Can you either create a new datanav.App class that subclasses dabo.dApp 
> (and have the generated App.py subclass that), or put the code into the 
> generated output for App.py?
> 
> Please remember that that generated code is intended to show people one 
> way to do things. Burying that one way inside Dabo itself doesn't help 
> people learn, does enforce a single way to do things, and doesn't 
> encourage them to modify it to suit their needs.

Got it.  given that it won't hurt to leave it for a few days, I'll leave it for 
now.  still haven't gotten form #1 'up' yet.  really need to get that done 
tonight.

> 
>> added code to create go.sh, a shell script that runs the main py script.
> 
> This shouldn't be needed (just run the main script). Perhaps a chmod 755 
> needs to happen on the generated file though. Also, it won't work on 
> Windows, and probably not Mac because pythonw is needed on that platform.
> 
> Also, you forgot to include the spec file for the go.sh file.

doh!  I bet that is why I committed the wrong file...

Carl

_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev

Reply via email to