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
click: init at 0.3.1 #38225
click: init at 0.3.1 #38225
Conversation
pkgs/top-level/all-packages.nix
Outdated
@@ -9570,7 +9570,7 @@ with pkgs; | |||
libantlr3c = callPackage ../development/libraries/libantlr3c {}; | |||
|
|||
libaom = callPackage ../development/libraries/libaom { }; | |||
|
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 you remove this diff?
buildInputs = stdenv.lib.optionals stdenv.isDarwin [ darwin.apple_sdk.frameworks.Security ]; | ||
|
||
meta = with stdenv.lib; { | ||
description = "The \"Command Line Interactive Controller for Kubernetes\""; |
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.
(non blocking) you could use """
instead of escaping "
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'm pretty sure that should be '' (two single quotes). (The triple quotes thing is Python.)
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.
yes, exactly, sorry:)
@mbode we generally don't put the commit id in the package name and the manual don't talk about this. |
a297e3b
to
f228c62
Compare
@nlewo I have removed the commit id. The next version is going to be released properly according to databricks/click#34. |
@mbode ok, thanks! |
Success on x86_64-linux (full log) Attempted: click Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: click Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: click Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: click Partial log (click to expand)
|
@mbode could you either fix the build for |
Motivation for this change
Add click.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)