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
vagrant_libvirt: init at 1.9.5 #26202
Conversation
@proteansec, thanks for your PR! By analyzing the history of the files in this pull request, we identified @cstrahan, @zimbatm and @abbradar to be potential reviewers. |
You need an override for the |
@Mic92 I tried adding the ruby-libvirt into the default.nix that you've mentioned, but it's not being picked up. You can see the output here:
Would you be kind enough to look at the commit and let me know if there's anything out of ordinary in there, so the ruby-libvirt dependency is not being passed the correct parameters. |
ruby-libvirt is now fixed. |
src = fetchurl { | ||
url = "https://github.com/mitchellh/vagrant/archive/v${version}.tar.gz"; | ||
sha256 = "0ancxh93ziyjak1wvjnbg2x7jv1brgbd4pxyi51zkdj867g8xsv9"; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it not possible to extend vagrant instead of packaging it again?
"--with-libvirt-lib=${libvirt}/lib" | ||
]; | ||
}; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this bit could already be merged regardless of the rest
I've modified the package,so the wrapper now provides the installed dependent gems to vagrant, while the build succeeds now. However, when running vagrant, it complains about
The error comes from
@zimbatm, building from source is a lot nicer than using the package that has already been build. Hopefully, we can merge the new version of vagrant (once working), which builds from source, and remove the old version from nixpkgs. @zimbatm @Mic92 Any ideas how to resolve the final issue, so we can finally make this work and possibly merge into nixpkgs. |
This has been superseded by the PRs in the references above. |
Motivation for this change
First of all, this is not ready to merge. Basically, I would like to package vagrant building it by source, but also enable vagrant-libvirt plugin. Hopefully, this will also provide the means to install other vagrant plugins easily.
The reason for adding additional flags (gemflags) is needed, because the libvirt dependency cannot be found otherwise. The libvirt is specified in buildInputs, but even though I've specified it in build inputs, I'm getting the following error when trying to build without the additional flags ("--with-libvirt-include=${libvirt}/include --with-libvirt-lib=${libvirt}/lib").
I'm having problems with the existing code, since nokogiri is unable to build, since it's missing the mini_portile2 dependency, even though it is properly specified and should be installed into the environment:
I would like to know if something has any ideas how to make this work.