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
snapraid: 11.1 -> 11.2 #35272
snapraid: 11.1 -> 11.2 #35272
Conversation
@GrahamcOfBorg build snapraid |
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
sha256 = "05hq1478k61pa1hzg6xjaw63z6nba3yd7k2fxsxpb1fw06j5ywmj"; | ||
# needed by git for determining the version number | ||
deepClone = true; | ||
leaveDotGit = true; |
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.
Uh, this is always ugly and often break the checksum. Is it possible to do a simple patch 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.
You can simple say VERSION=version;
in nix to make it a environment variable: https://github.com/amadvance/snapraid/blob/f31dfada3e52e03487d4f68cfacb3b984381385b/autover.sh#L14
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.
You are right. This is a much better choice.
f47baf1
to
ac64f9c
Compare
Motivation for this change
Update
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)cc @makefu