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
tanka: init at v0.11.1 #94659
tanka: init at v0.11.1 #94659
Conversation
Yessir ! I never saw that. But, I will in the future !
Le mar. 4 août 2020 18:21, Timothy Stott <notifications@github.com> a
écrit :
… ***@***.**** commented on this pull request.
------------------------------
In pkgs/applications/networking/cluster/tanka/default.nix
<#94659 (comment)>:
> @@ -0,0 +1,23 @@
+{ buildGoModule, fetchFromGitHub, lib }:
+
+buildGoModule rec {
+ pname = "tanka";
+ version = "0.11.1";
+
+ src = fetchFromGitHub {
+ owner = "grafana";
+ repo = pname;
+ rev = "v${version}";
+ sha256 = "0hp10qgalglsdhh6z6v4azh2hsr89mdrv1g5lssfl5jyink409yd";
+ };
+
+ vendorSha256 = "15x8fqz2d2793ivgxpd9jyr34njzi1xpyxdlfyj1b01n2vr3xg4m";
+
🙋 Could we please set the package version main.Version=${version} via
buildFlagsArray.
As an example the gopass package:
https://github.com/NixOS/nixpkgs/blob/e3cba5aad991a0f355545fc0b069cbf987b577d0/pkgs/tools/security/gopass/default.nix#L25
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#94659 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHO2FHDQBZE5LXUE3PWBB3R7CCU7ANCNFSM4PUMTNOQ>
.
|
I will change it as soon I join my computer I mean
Le mar. 4 août 2020 18:43, Michael Faille <michael.faille@gmail.com> a
écrit :
… Yessir ! I never saw that. But, I will in the future !
Le mar. 4 août 2020 18:21, Timothy Stott ***@***.***> a
écrit :
> ***@***.**** commented on this pull request.
> ------------------------------
>
> In pkgs/applications/networking/cluster/tanka/default.nix
> <#94659 (comment)>:
>
> > @@ -0,0 +1,23 @@
>
> +{ buildGoModule, fetchFromGitHub, lib }:
>
> +
>
> +buildGoModule rec {
>
> + pname = "tanka";
>
> + version = "0.11.1";
>
> +
>
> + src = fetchFromGitHub {
>
> + owner = "grafana";
>
> + repo = pname;
>
> + rev = "v${version}";
>
> + sha256 = "0hp10qgalglsdhh6z6v4azh2hsr89mdrv1g5lssfl5jyink409yd";
>
> + };
>
> +
>
> + vendorSha256 = "15x8fqz2d2793ivgxpd9jyr34njzi1xpyxdlfyj1b01n2vr3xg4m";
>
> +
>
>
> 🙋 Could we please set the package version main.Version=${version} via
> buildFlagsArray.
>
> As an example the gopass package:
>
> https://github.com/NixOS/nixpkgs/blob/e3cba5aad991a0f355545fc0b069cbf987b577d0/pkgs/tools/security/gopass/default.nix#L25
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#94659 (review)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAHO2FHDQBZE5LXUE3PWBB3R7CCU7ANCNFSM4PUMTNOQ>
> .
>
|
f5696ff
to
2b78b5c
Compare
Result of 1 package built:- tanka |
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 think this can be enabled for all Unix platforms.
Also, it seems that the tk
command supports bash completions. Would be nice to add them as well. Something like the following in postInstall
should work (untested):
echo "complete -C $out/bin/tk tk" > tk.bash
installShellCompletion tk.bash
(And add installShellFiles
to nativeBuildInputs
).
homepage = "https://github.com/grafana/tanka/"; | ||
license = licenses.asl20; | ||
maintainers = with maintainers; [ mikefaille ]; | ||
platforms = platforms.linux ++ platforms.darwin; |
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.
Any reason to suspect that this doesn't work on other Unixes?
platforms = platforms.linux ++ platforms.darwin; | |
platforms = platforms.unix; |
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.
BSD systems works for Nix ?
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.
Some people are working on it. Once there is more mature support, it would be nice to have these derivations (that probably work just fine) work immediately.
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 would be a pleasure to support BSD :D
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.
Done !
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.
LGTM
Result of nixpkgs-review pr 94659
1
1 package built:
- tanka
Would by nice to have shell completion as suggested in my other comment. Maybe you can open another PR for that?
@danieldk Yes, I would try, It seems it injects |
Oh, I just seen the comment above. |
Motivation for this change
Tanka is a tool built by Grafana and Redhat to facilitate Kubernetes assets on jsonnet format.
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)