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

Dependency on puppetlabs-ruby constrains use #49

Closed
dcj opened this issue Jun 1, 2014 · 8 comments
Closed

Dependency on puppetlabs-ruby constrains use #49

dcj opened this issue Jun 1, 2014 · 8 comments

Comments

@dcj
Copy link

dcj commented Jun 1, 2014

The puppetlabs-ruby module only supports Debian and Redhat based systems, so when I attempt to use this r10k module on another OS, (e.g. OS X/Darwin), it fails.

It would be a nice enhancement if there was a way to use this r10k module on a broader set of machines....

@igalic
Copy link
Contributor

igalic commented Jun 2, 2014

@dcj the way to go is to fix those modules then ;)

@dcj
Copy link
Author

dcj commented Jun 2, 2014

@igalic Agreed, but it my current impression that puppet labs only supports specific OSes with their PE product, and these modules are intended to support only that product, and therefore they aren't interested in additions for other OSes.

@igalic
Copy link
Contributor

igalic commented Jun 3, 2014

That is not entirely true. While puppetlabs is highly focused on getting support for PE supported platforms into supported modules, that doesn't mean that they will leave the FOSS community by the side.

If you can update puppetlabs-ruby to support your favourite OS, and include Documentation and spec tests / acceptance tests (if possible / necessary), there is no reason why that shouldn't get merged.

@acidprime
Copy link
Collaborator

Typically they will take any well thought out code they can test against

@acidprime acidprime reopened this Jun 3, 2014
@acidprime
Copy link
Collaborator

I would be interested in getting this module working on darwin though, so please file any thing thats not working.

@acidprime
Copy link
Collaborator

Seems like the ignore option in this PR #67 might solve this in a round about fashion

@acidprime
Copy link
Collaborator

also now that puppet is supported on Mac OS X pe_gem should just work

@acidprime
Copy link
Collaborator

closing this now that #67 is merged in v2.3.0

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

No branches or pull requests

3 participants