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
check_ssl_cert: init at 1.51.0 #28069
Conversation
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 don't know why @qknight merged this within a minute of opening, I think this needs some changes
name = "check_ssl_cert-${version}"; | ||
version = "1.51.0"; | ||
|
||
src = fetchgit { |
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.
Should use the dedicated fetchFromGitHub
|
||
src = fetchgit { | ||
url = https://github.com/matteocorti/check_ssl_cert; | ||
rev = "4e31a82008a48cd6efb82914e0bb1e136b339a31"; # v1.51.0 |
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.
rev = "v${version}"
will do too
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.
awesome, didn't know that! your solution is so much better.
}; | ||
|
||
buildInputs = [ makeWrapper file ]; | ||
#nativeBuildInputs = [ makeWrapper ]; |
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.
Should be removed, although I don't actually know where makeWrapper should be
@@ -0,0 +1,33 @@ | |||
{ pkgs, openssl }: | |||
#with import <nixpkgs> {}; |
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.
Should be removed
Repushed in fd6e51d, sorry for the confusion |
Thanks. Any more action I have to take? |
Nope fine now |
thanks for the help everyone, sorry i didn't see the issues you had to fix! |
Added package for nagios plugin check_ssl_cert