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
libvmi: init at 3e4114 #26162
libvmi: init at 3e4114 #26162
Conversation
sha256 = "0vbmrj0ij19i55afkqj64q7sgh0scpwk3c99qx6p6gn1qcy2wdss"; | ||
}; | ||
|
||
buildInputs = [ glib xen which yacc bison flex libtool autoconf automake pkgconfig libvirt ]; |
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.
- buildInputs = [ glib xen which yacc bison flex libtool autoconf automake pkgconfig libvirt ];
+ buildInputs = [ glib xen which libvirt ]; # probably which too.
+ nativeBuildInputs = [ yacc bison flex libtool autoconf automake pkgconfig ];
|
||
stdenv.mkDerivation rec { | ||
name = "libvirt-${version}"; | ||
version = "20170527-${stdenv.lib.strings.substring 0 7 rev}"; |
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.
we prefer to have a numeric value at the end, to make it sortable for nix-env -u
. See also https://nixos.org/nixpkgs/manual/#sec-package-naming
nativeBuildInputs = [ yacc bison flex libtool autoconf automake pkgconfig ]; | ||
|
||
preConfigure = '' | ||
./autogen.sh |
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.
use autoreconfHook
instead?
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.
Thanks for the tip, fixed.
with stdenv.lib; | ||
|
||
stdenv.mkDerivation rec { | ||
name = "libvirt-${version}"; |
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.
Should be libvmi.
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)