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
kind: init at 2ae73f8e #47967
kind: init at 2ae73f8e #47967
Conversation
Success on aarch64-linux (full log) Attempted: kind Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: kind Partial log (click to expand)
|
goPackagePath = "sigs.k8s.io/kind"; | ||
excludedPackages = "images/base/entrypoint"; | ||
|
||
outputs = [ "bin" "out" ]; |
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 is the default I think.
@@ -0,0 +1,28 @@ | |||
{ stdenv, lib, 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.
lib
is not used
|
||
buildGoPackage rec { | ||
name = "kind-${version}"; | ||
version = "20181003-${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.
According to the manual, the date format is YYYY-MM-DD
.
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.
So it is 20181003 = 2018 October 3, the day of referenced commit
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.
Hmm, not sure to understand:/
In fact, I was talking about missing dashes: s/20181003/2018-10-03
.
Also, I think the date is more the date of the day you are writing the package since the goal is to have an ordered package name. But in this PR, 10-03
is fine.
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.
Ok i understand, i fixed it.
meta = { | ||
description = "Kubernetes IN Docker - local clusters for testing Kubernetes"; | ||
homepage = https://github.com/kubernetes-sigs/kind; | ||
maintainers = with stdenv.lib.maintainers; [ offline ]; |
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.
Could be shortened to with maintainers...
6d140f5
to
e2376d5
Compare
I have fixed other things, except for one thing noted above, thanks! |
Success on aarch64-linux (full log) Attempted: kind Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: kind Partial log (click to expand)
|
e2376d5
to
6124447
Compare
Success on aarch64-linux (full log) Attempted: kind Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: kind Partial log (click to expand)
|
Thanks! |
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)