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
wuzz: init at 2017-02-05 #22464
wuzz: init at 2017-02-05 #22464
Conversation
|
||
buildGoPackage rec { | ||
name = "wuzz"; | ||
version = "89bebba7a4a00c52e97d411a3e82660189adc4ff"; |
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.
Git revisions are invalid versions; please use the commit date in ISO 8601 format.
@@ -0,0 +1,25 @@ | |||
{ stdenv, lib, pkgconfig, buildGoPackage, fetchFromGitHub }: |
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.
pkgconfig
is unused.
@joachifm Thank you for the review. I have updated the pull request as per your suggestions. |
{ stdenv, lib, buildGoPackage, fetchFromGitHub }: | ||
|
||
buildGoPackage rec { | ||
name = "wuzz"; |
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 version must be embedded in the name, as in wuzz-${version}
. The version
attribute is not special; version strings are actually extracted from name
using parseDrvName
.
It's still needed by KDE4. It was there, just unreferenced from all-packages.
@pradeepchhetri you can also force-push the PR branch to fixup such issues |
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/
)