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

Remove support for providers yum, zypper #489

Merged
merged 2 commits into from
May 26, 2019
Merged

Remove support for providers yum, zypper #489

merged 2 commits into from
May 26, 2019

Conversation

dhoppe
Copy link
Member

@dhoppe dhoppe commented May 10, 2019

This pull request removes support for providers yum and zypper. We also might want to remove support for OpenBSD and remove the package resource, since this was introduced for Puppet Enterprise which has r10k already installed.

This will fix issue #449.

@alexjfisher
Copy link
Member

@dhoppe
Copy link
Member Author

dhoppe commented May 10, 2019

@alexjfisher Hm. I missed that somehow and will drop the latest commit. Interesting that SLES provides a package and RHEL does not. Usually it is the other way around.

@rnelson0
Copy link
Member

r10k 2.3.0 was released on May 18, 2016 (current is 3.2.0 from April 16, 2019), so I'm not sure that I would say the packages are anything near recent, even if the timestamps are.

@dhoppe
Copy link
Member Author

dhoppe commented May 16, 2019

Someone needs to make a decision. I still think we should remove yum and zypper, because it does not hurt to install a gem.

@bastelfreak
Copy link
Member

let's keep it open for 3 more days. I'm 👍 for dropping it. If nobody offers an argument to keep it, we can drop it.

@dhoppe dhoppe changed the title [WIP] Remove support for providers yum, zypper Remove support for providers yum, zypper May 20, 2019
@bastelfreak bastelfreak merged commit 13279bc into voxpupuli:master May 26, 2019
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.

4 participants