[sugar] [PATCH] Bundlebuilder use manifest, fix_manifest function

Marco Pesenti Gritti mpgritti at gmail.com
Sun Jun 8 05:34:18 EDT 2008


On Sun, Jun 8, 2008 at 3:51 AM, Jameson Chema Quinn
<jquinn at cs.oberlin.edu> wrote:
> Sorry, Marco, but here is yet another version of the patch. I decided to
> refactor the Config class out of bundlebuilder - all it was carrying in the
> common start() case was bundle_name; and as for other cases, it is simpler
> to call Builder(...params...) than Builder(Config(...params...)).

I'm using a config object because it's easier to carry around in the
other Xer. I don't think having to create the Config is a big deal,
calling it directly is not the primary use case anyway. But i you
*really* feel that we need this bit of convenience, we can add a
create_builder() which setups the config for you.

Marco


More information about the Sugar mailing list