Skip to content
This repository was archived by the owner on Jan 15, 2024. It is now read-only.

Fixing dependency for rubygems#178

Merged
Ma27 merged 1 commit intowilldurand:masterfrom
jwatroba:patch-1
Jan 23, 2018
Merged

Fixing dependency for rubygems#178
Ma27 merged 1 commit intowilldurand:masterfrom
jwatroba:patch-1

Conversation

@jwatroba
Copy link
Copy Markdown
Contributor

Overview

Fixes: Error: /Stage[main]/Nodejs::Instance::Pkgs/Package[semver]: Provider gem is not functional on this host

The semver gem requires rubygems to be installed.

Related issues

Fixes: Error: /Stage[main]/Nodejs::Instance::Pkgs/Package[semver]: Provider gem is not functional on this host

The semver gem requires rubygems to be installed.
@willdurand willdurand requested a review from Ma27 January 11, 2018 01:18
@Ma27
Copy link
Copy Markdown
Collaborator

Ma27 commented Jan 22, 2018

@jwatroba thanks for your PR! Unfortunately my time is quite limited ATM, however I didn't forget about this.
I'll try to have a look at this in one of the next days :)

@Ma27
Copy link
Copy Markdown
Collaborator

Ma27 commented Jan 23, 2018

merging for now. We might want to drop semver in the future as it's only needed on the puppet master (the entire package managing is currently optional and can be disabled using build_deps => false), however I think this is the best way to fix it for now.

@Ma27 Ma27 merged commit 55a46f9 into willdurand:master Jan 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants