runningman/puppet-network — pull request #2 by hakamadare (2012-03-01 20:04:38+00:00)
Title: Update for compatibility with recent Puppet releases

Hi! I tried out your module on some recent Puppet releases (2.7.1 master, 2.6.4 client), and saw a bunch of syntax errors. Digging into the code, I found that the master wouldn't compile your manifests because you had your defines inside of conditionals.

I ended up splitting out each of the defines into their own files, so that module autoloading could load them all, and you wouldn't need any more 'include "*.pp"' directives.

Let me know what you think of this. As part of the project I'm working on, I'm also planning on adding code to enable this module to work on Debian/Ubuntu systems, so keep an eye out for another pull request sometime later.

-steve