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
lego: init at unstable-2018-02-02 #34830
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.
Just some minor comments
pkgs/tools/admin/lego/default.nix
Outdated
|
||
buildGoPackage rec { | ||
name = "lego-${version}"; | ||
version = "0.4.2-pre-${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.
Since this revision is not released, can you please use for the version, the format unstable-YYYY-MM-DD
as specified in https://nixos.org/nixpkgs/manual/#sec-package-naming
pkgs/tools/admin/lego/default.nix
Outdated
version = "0.4.2-pre-${lib.strings.substring 0 7 rev}"; | ||
rev = "06a8e7c475c03ef8d4773284ac63357d2810601b"; | ||
|
||
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.
Can you please use fetchFromGitHub
pkgs/tools/admin/lego/default.nix
Outdated
|
||
goPackagePath = "github.com/xenolf/lego"; | ||
goDeps = ./deps.nix; | ||
subPackages = [ "." ]; |
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 useless I guess
@nlewo - Thanks for the prompt feedback! Just pushed another commit with my updates; do you want me to squash it into the original 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.
Thanks for your changes. You can squash your commits.
b564794
to
ac5e4d4
Compare
@nlewo - Done! |
@GrahamcOfBorg build lego |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-darwin (full log) Partial log (click to expand)
|
@adisbladis - Thank you very much! 😀 |
@andrew-d Thank you :) |
Motivation for this change
Adding lego, a Let's Encrypt client that supports DNS-based challenges with a variety of providers.
Dependencies were generated with go2nix today (2018-02-10).
Things done
build-use-sandbox
innix.conf
on non-NixOS)Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)Tested compilation of all pkgs that depend on this change usingnix-shell -p nox --run "nox-review wip"
./result/bin/
)