Builders should be defined as instances, not as a dictionary

[warner]

The c['builders'] specification is the last one in master.cfg that is defined with a dictionary. I think it would be much clearer if it were defined as an instance. So instead of:

    b1 = {"name": "svn-hello",
          "slavename": "bot1",
          "builddir": "svn-hello",
          "factory": factory.BuildFactory(steps),
          }
    c['builders'].append(b1)

you would do:

    b1 = Builder("svn-hello", "bot1", factory.BuildFactory(steps))
    Builders.append(b1)

(assuming that #9 was implemented)

Also, the "builddir" argument should go away, and just use the builder name. Having to specify a separate directory is annoying. It might be nice to add a separate "verbose builder name" argument, though, since the normal builder name gets used as a directory name and thus can't have slashes (and really doesn't want to have spaces).

[warner]

no progress on this yet, bumping to 0.7.8

[warner]

After finishing this, implement #119 (builder descriptions)

[warner]

also see #148

[warner]

not happening for 0.7.8, pushing it out.

[marcusl]

Marked #148 as duplicate of this.

[dustin]

This really should get made into an object for the next release - that makes it much easier to add default arguments, etc.

[dustin]

I'm working on this now -- just rewriting all of the unit tests and the docs.

[dustin]

commit a70a00b22c920ae2c1e95f7bc614a7a94ded861a
Author: Dustin J. Mitchell <dustin@zmanda.com>
Date:   Sat Dec 26 12:13:35 2009 -0600

    (refs #10) add BuilderConfig class, use it in tests, and doc it

[Dustin J. Mitchell]

use BuilderConfig? in sample.cfg

fixes #923, fixes #10

Changeset: ca9eaece77d2b8c6f3519f981ef69c64b68687a4