Include concat::setup from concat, so users don't need to

[djmitche]

This includes concat::setup directly from concat, so users don't have to. Of course, it won't hurt to include it manually, too.

This should fix #gh-16.

I'm not sure what @dalen meant in his last comment there, about resource defaults. I'll revise if you can clarify.

[ripienaar]

likely cause hassles in future puppet but will deal with that when it happens

thanks

[dalen]

I meant that the line:
$group = $concat::setup::root_group,

in the parameters section of concat requires concat::setup to be parsed already. so it should really inherit from concat::setup using the style mentioned at http://docs.puppetlabs.com/guides/style_guide.html#class-parameter-defaults that requires 2.6.3 or later. But I'm not sure if you want to break 2.6.2 compatibility just yet as it is the version still used in debian stable for example.

[ripienaar]

yes, wouldnt want to break backwards compat with 2.6.x

[djmitche]

Ah, I didn't spot that. Could you set the default to a sentinel value, and substitute it in the class itself (where the value is available)?