Ticket #148 (closed enhancement: duplicate)

Opened 1 year ago

Last modified 2 months ago

Add 'Builder' class for master config

Reported by: stefan Assigned to:
Priority: minor Milestone: undecided
Component: configuration Version: 0.7.6
Keywords: Cc:

Description

Right now the master config file expects builders to be declared as dictionaries. I'd like to suggest to replace those by actual class instances (say, 'Builder'), to make the config file more expressive, as well as more robust (the Builder constructor can check all required keywords are present).

In my master.cfg, I do:

  
  class Builder(dict):

    def __init__(self, **kwds):

      self.update(kwds)


  c['builders'] = [Builder(name='sdist', slave='cugel', builddir='sdist',
                           factory=BuildFactory([a, b, c]))]

which works nicely.

Change History

04/28/08 13:34:01 changed by warner

Yeah, I'd like to move to using instances instead of dicts for builder configuration.. see #10 .

11/14/08 01:02:11 changed by marcusl

  • status changed from new to closed.
  • resolution set to duplicate.

This is actually a duplicate of #10