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

fixes #14225 - add missing require for Resolv #3332

Closed
wants to merge 1 commit into from

Conversation

domcleal
Copy link
Contributor

Setting itself is explicitly required too, which will provide a better
error message in future for this and similar issues loading the class.

Setting itself is explicitly required too, which will provide a better
error message in future for this and similar issues loading the class.
@ares
Copy link
Member

ares commented Mar 16, 2016

I'm having issues to reproduce the problem. I installed clean wheezy and ran foreman-installer (without any other options). Foreman loads just fine. The code looks sane though. I checked production.log from the CI task but I don't see anything mentioned in the issue, do you have your own reproducer?

@domcleal
Copy link
Contributor Author

@ares which build of Foreman? Note that the nightly builds on deb.theforeman.org are the ones that passed tests, the one that failed with this error didn't - you can find it on stagingdeb.theforeman.org. Check that 7339369 is applied.

@ares
Copy link
Member

ares commented Mar 17, 2016

Ah makes sense, I was able to reproduce with packages from staging. Merged as f7f6bd5, thanks @domcleal!

@ares ares closed this Mar 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants