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
acme-client: init at 0.1.6 #33996
acme-client: init at 0.1.6 #33996
Conversation
|
||
src = fetchurl { | ||
url = "https://kristaps.bsd.lv/acme-client/snapshots/acme-client-portable-${version}.tgz"; | ||
sha512 = "730c20bdf9d72b24e66c54b009a282e04da3ea8ce3b9eb053750672c53c9586b2879d87a565ddbab033d7ba6a577dd6399313b20cf654b185905db4de988b6b7"; |
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.
Nix format (Base32) is preferred. Also, sha256
should be sufficient.
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.
Grabbed it from upstream's download page; will change
|
||
meta = { | ||
homepage = https://kristaps.bsd.lv/acme-client/; | ||
description = "Secure acme/Let's Encrypt client"; |
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.
s/acme/ACME/
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 copied verbatim from upstream's description, but happy to change.
meta = { | ||
homepage = https://kristaps.bsd.lv/acme-client/; | ||
description = "Secure acme/Let's Encrypt client"; | ||
platforms = stdenv.lib.platforms.all; |
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.
If meta
had with stdenv.lib
, it could have been omitted on individual attributes:
meta = with stdenv.lib; {
platforms = platforms.all;
license = licenses.isc;
}
Also, it seems unlikely to me that it would run on Windows: probably you've meant 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.
Great review, appreciate the careful look! Will make all those changes.
@@ -0,0 +1,36 @@ | |||
{ stdenv | |||
, apple_sdk |
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.
Given that this attribute is only required on Darwin, it could be optional:
, apple_sdk ? null
@yegortimoshenko your comments have been addressed, thanks! |
@GrahamcOfBorg build acme-client |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Motivation for this change
Add acme-client package.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)