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
f3: newer git rev, build extra binaries #20724
Conversation
tested binaries and worked for me.Thanks! 👍 |
{ stdenv, fetchFromGitHub }: | ||
{ stdenv, fetchFromGitHub | ||
, parted, udev | ||
}: | ||
|
||
stdenv.mkDerivation rec { | ||
name = "f3-${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.
this should be then also rewritten as:
f3-unstable-2016-11-16
Okay, done; was wondering about that but wasn't sure if it needed to be numerically greater than |
Nix does not really care about version number. It is just about conventions. |
Thanks! |
I thought ... I looked through the nix source and I'm pretty sure starting the first version component (delimited by |
I needed to make a new PR: #20751 |
Motivation for this change
The
v6.0
tag is almost a year old, so I've just switched to the newest revision on github. Also, 3 binaries (f3brew, f3fix, f3probe) weren't being built by this derivation. I had to add a HACK for building with the current gcc+glibc, as described here: AltraMayor/f3#34Things 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/
)