Eirik Meland wrote:
>> Looks good and is working, thanks.
>
> Added to cvs?

Yes, just added it with some small modifications.

> If you are thinking of that the .ini format is yet another format
> and that it would have been better to configure the log using
> python, then I agree. Doing this is no problem, but I still think
> that having the log-configuration in a separate file is a good idea.

I just checked it in without this file.

Developers please read:

o The logging stuff is set in sysconfig.py and not in a special
  file. So every helper importing sysconfig has the logger set up.

o The old _debug_ is gone. I also want to remove config.DEBUG, so if
  someone has some time, please port more of the code. I also want to
  remove _all_ print statements except for helpers, we should use
  logging. 

o The root logging level is WARNING. This means you only see warnings,
  no info and no debug. To set the level to INFO add the following
  line to local_conf.py:
  logging.getLogger().setLevel(logging.INFO)
  This is more or less identical with the old DEBUG = 1. To see the
  whole debug (old DEBUG = 2) use the level logging.DEBUG.

o Some parts of freevo use a different logger than the root
  logger. Examples are all subtypes like video and childapp s special
  case for running programs. If you only want some childapp debug add
  the following line
  logging.getLogger('childapp').setLevel(logging.INFO)
  I'm also planing to add special logger for memory debug (__del__)
  and timing (how long does action xy take). 

o We should port all debug to the new logger. While we are doing this,
  maybe remove some debug we don't need anymore. 


Comments please


Dischi

-- 
If windows is the answer, it must have been a stupid question.

Attachment: pgpPIYPCFxKS2.pgp
Description: PGP signature

Reply via email to