[Buildbot-commits] [Buildbot] #960: Convert all buildbot classes to new style, i.e. inherit from object.

Buildbot buildbot-devel at lists.sourceforge.net
Tue Nov 2 21:24:39 UTC 2010


#960: Convert all buildbot classes to new style, i.e. inherit from object.
-------------------+--------------------------------------------------------
Reporter:  maruel  |       Owner:       
    Type:  task    |      Status:  new  
Priority:  major   |   Milestone:  0.8.+
 Version:  0.8.1   |    Keywords:  tests
-------------------+--------------------------------------------------------
Changes (by dustin):

  * keywords:  tests, ship-this => tests
  * milestone:  0.8.3 => 0.8.+


Comment:

 This fails badly when unpickling builds from previous versions.  Comments
 on the commit -
 http://github.com/catlee/buildbot/commit/0593a9dbff1ccc286b06cbbcac3408e7d996b4ae#commitcomment-183550.

 One option is to write some sort of conversion code into the unpickling.
 The other option is to wait for the StatusDB when we're not using pickles
 anymore. I prefer the latter, honestly..

 Also, I'm not so keen on tests looking at the sizes of the classes before
 and after.  It's nice to demonstrate that this has a savings and so on,
 but it doesn't really constitute a unit test - more of a performance test,
 I guess.

-- 
Ticket URL: <http://buildbot.net/trac/ticket/960#comment:6>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the Commits mailing list