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
kvmtool: Init at 0.20200424 #86174
kvmtool: Init at 0.20200424 #86174
Conversation
sha256 = "0ir6aqvipss145r85324nw84k4ilzxa2rsa1wr5303p1h6gs0qkk"; | ||
}; | ||
|
||
makeFlags = [ "DESTDIR=$(out)" "prefix=/" ]; |
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.
I assume it tries to install something outside of prefix
? We usually we should avoid prefix=
as it can lead to wrong references within the package.
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.
It doesn't try to install outside of prefix. It oddly uses $HOME
as its default prefix
, which causes a homeless-shelter
path component in the installed path. Default DESTDIR
is empty.
Oh, I see that I don't need both DESTDIR and prefix -- just prefix=$(out)
also works.
makeFlags | Outcome |
---|---|
(nothing) | cannot create directory '/homeless-shelter': Permission denied |
DESTDIR=$(out) |
Executable is result/homeless-shelter/bin/lkvm (bad) |
DESTDIR=$(out) prefix=/ |
Executable is result/bin/lkvm (good) |
DESTDIR=$(out) prefix= |
Executable is result/bin/lkvm (good) |
prefix=$(out) |
Executable is result/bin/lkvm (good) |
All three of the entries labelled "(good)" produce the same binary (hashes to 3c896d01...).
Which is preferred?
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.
The latter one: prefix=$(out)
.
I notice that this is failing on aarch64 because it can't find So this is not ready to merge. I'll sort this out and try again later. Thank you, Mic92, for the review. |
Motivation for this change
Make kvmtool available.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)This project doesn't really do releases or version numbers. This is the latest git commit, with the git committer date as version number, adopting the same convention that other package repositories are using: https://repology.org/project/kvmtool/versions