[Dev] application.Util.py status

[Mike Taylor]

John,

One of the motivating factors for me to suggest placing the option  
parsing routine into Util.py is that I envision that it will be mostly  
helper code to load values from the four possible sources:  default  
base values, chandler config file, environment variables and/or command  
line values.  All of the decisions based on the values should continue  
to be in Application.py -- sorry if I didn't make that clear in my  
original post.

I have a nice helper routine that I've used in my other python projects  
that takes a number of dictionaries and returns a dictionary of values  
so the messy stuff is nicely tucked away and Application.py doesn't get  
bloated.

---
Bear
http://code-bear.com
PGP Fingerprint = 9996 719F 973D B11B E111  D770 9331 E822 40B3 CD29


On Dec 8, 2004, at 12:06 PM, John Anderson wrote:

> Startup option parsing should probably go in Application.py if it's  
> part of Chandler startup.
>
> The startup stuff in Util.py is there only for unit tests, and I need  
> to make a pass to simplify it and modify Application.py to also use  
> it. I'll try to do it soon.
>
> Also, I'd be happy to review any changes you make to Application.py,  
> since I wrote most of it.
>
> John
>
> Mike Taylor wrote:
>
>> I'm going thru the Chandler startup process to see what is new and I  
>> have a question :)
>>
>> The routines in application/Util.py seem to be duplicated by code  
>> inside of application/Application.py - is there a migration in  
>> progress *to* Util.py or ???
>>
>> If it's not a migration, then is there a need for Util.py.  Should I  
>> start looking into finishing the migration if it is?
>>
>> The reason for me doing this is that I need to add some startup  
>> option parsing code to allow me to continue with the linux installer  
>> and, while scouting the best place for the new code, I came across  
>> Util.py which seems to be a good spot for the code.
>>
>> ---
>> Bear
>> http://code-bear.com
>> PGP Fingerprint = 9996 719F 973D B11B E111  D770 9331 E822 40B3 CD29
>>
>> ---------------------------------------------------------------------- 
>> --
>>
>> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>>
>> Open Source Applications Foundation "Dev" mailing list
>> http://lists.osafoundation.org/mailman/listinfo/dev
>>
> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>
> Open Source Applications Foundation "Dev" mailing list
> http://lists.osafoundation.org/mailman/listinfo/dev
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
Url : http://lists.osafoundation.org/pipermail/dev/attachments/20041209/d5f1ff4d/PGP.bin