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

Librarian-puppet errors resolving dependencies for puppet concat #37

Merged
merged 1 commit into from
Jan 28, 2016

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Jan 28, 2016

Errors trying to install this module with librarian:

 Module puppetlabs-apt found versions: 2.2.1, 2.2.0, 2.1.1, 2.1.0, 2.0.1, 2.0.0, 1.8.0, 1.7.0, 1.6.0, 1.5.2, 1.5.1, 1.5.0, 1.4.2, 1.4.0, 1.3.0, 1.2.0, 1.1.1, 1.1.0, 1.0.1, 1.0.0, 0.0.4, 0.0.3
[Librarian]     Checking puppetlabs-apt/2.2.1 <https://forgeapi.puppetlabs.com>
[Librarian]       Resolved puppetlabs-apt (< 3.0.0, >= 1.8.0) <(no source specified)> at puppetlabs-apt/2.2.1 <https://forgeapi.puppetlabs.com>
[Librarian]   Resolved puppetlabs-apt (< 3.0.0, >= 1.8.0) <(no source specified)>
[Librarian] Conflict between puppetlabs-concat (< 2.0.0, >= 1.1.0) <(no source specified)> and puppetlabs-concat/2.1.0 <https://forgeapi.puppetlabs.com>

@ehelms
Copy link
Member Author

ehelms commented Jan 28, 2016

@ekohl I wondered if anything struck you about this? If there was a better solution.

@ekohl
Copy link
Member

ekohl commented Jan 28, 2016

Oh, I see they released 2.1.0. I think something pinned it to < 2.0.0 and librarian isn't good enough in conflict resolution. That's a shame.

@ehelms
Copy link
Member Author

ehelms commented Jan 28, 2016

So this looks like an OK change?

@ekohl
Copy link
Member

ekohl commented Jan 28, 2016

ACK I'm afraid so. I do wonder what happens when one dependency does want concat 2.1+ because that might cause the same problems but for now this moves us forward.

@ehelms
Copy link
Member Author

ehelms commented Jan 28, 2016

Yea, I wonder that too, I tested in a Puppetfile with a bunch of our other dependencies and it was OK. Thanks @ekohl

ehelms added a commit that referenced this pull request Jan 28, 2016
Librarian-puppet errors resolving dependencies for puppet concat
@ehelms ehelms merged commit 90a7105 into theforeman:master Jan 28, 2016
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.

None yet

2 participants