[Yum-devel] Re: [yum-cvs] yummain.py

seth vidal skvidal at fedoraproject.org
Fri Dec 14 14:18:46 UTC 2007


On Fri, 2007-12-14 at 05:10 -0500, Florian Festi wrote:
> yummain.py |  110 +++++++++++++++++++++++++++++++++++++------------------------
>  1 file changed, 67 insertions(+), 43 deletions(-)
> 
> New commits:
> commit 60b388ac8d47ae69bc7d60b23354272e8d70b4d1
> Author: Florian Festi <ffesti at redhat.com>
> Date:   Thu Dec 13 13:49:10 2007 +0100
> 
>     Support profiling for yum as a whole
> 


Florian,
 Why did you check this in? I don't think we had finished discussing it.
I don't like this 'email a patch on day 1, if no comments say NOOOOO
then commit it' policy you seem to have taken up. I agree with what Tim
said yesterday that there's no compelling reason to put this in yummain
at all when we can, instead just create a separate yumprof.py which
duplicates yummain.py which we can use for running the profiling
routines instead.

-sv





More information about the Yum-devel mailing list