Ticket #148 (closed enhancement: duplicate)

Opened 5 years ago

Last modified 5 years ago

Add 'Builder' class for master config

Reported by: stefan Owned by:
Priority: minor Milestone: undecided
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

comment:1 Changed 5 years ago by warner

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

comment:2 Changed 5 years ago by marcusl

  • Status changed from new to closed
  • Resolution set to duplicate

This is actually a duplicate of #10

Note: See TracTickets for help on using tickets.