jedi4ever/veewee — pull request #344 by mconigliaro (2012-07-23 17:29:30+00:00)
Title: Merge options from Veeweefile

We discovered evidence that it should be possible to configure Veewee via a "Veeweefile," but since we couldn't find any documentation on this particular feature (nor could we get it to work "out of the box"), we basically had to reverse-engineer it.

The Veeweefile appears to expect a format like this:

```
Veewee::Config.run do |config|
config.veewee.
=
end
```

Although the file above will get parsed without errors, we could not get Veewee to honor any of the options set within the Veeweefile. `Veewee::Environment#initialize` does not seem to read the Veeweefile on its own. Interestingly the call to env.load! in https://github.com/jedi4ever/veewee/blob/master/bin/veewee#L15 gives us what we want, but https://github.com/jedi4ever/veewee/blob/master/lib/veewee/command/virtualbox.rb#L14-18 (for example) is missing the crucial call to `venv.load!`, which means you will just end up with the default options every time.

Rather than adding `venv.load!` all over the place, I decided to patch `Veewee::Environment#initialize` so that the Veeweefile options get merged in right from the beginning. I hope this was the right thing to do...