Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed class containment issue in "init.pp" #5

Merged
merged 1 commit into from
Sep 8, 2014
Merged

Fixed class containment issue in "init.pp" #5

merged 1 commit into from
Sep 8, 2014

Conversation

BackSlasher
Copy link
Contributor

Because of Class Containment, the require => Class['composer'] in composer::project didn't work (composer would try to run before being installed).
Containment implemented using anchor pattern, because already dependent on stdlib

Because of [Class Containment](http://puppetlabs.com/blog/class-containment-puppet), the `require => Class['composer']` in `composer::project` didn't work (composer would try to run before being installed).
Containment implemented using [anchor pattern](http://projects.puppetlabs.com/projects/puppet/wiki/Anchor_Pattern), because already dependent on stdlib
@igalic
Copy link
Contributor

igalic commented Sep 8, 2014

thanks again. this looks better.
(it's a little unfortunate that we cannot just assume everyone is running the latest version of puppet :(

(would also be nice to find out why rspec can't recognize composer as being included, or perhaps just drop that test)

igalic added a commit that referenced this pull request Sep 8, 2014
Fixed class containment issue in "init.pp"
@igalic igalic merged commit 3adf56b into Brainsware:master Sep 8, 2014
@BackSlasher BackSlasher deleted the add-anchor branch September 8, 2014 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants