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
certbot: add certbot.withPlugins #92786
Conversation
dd04248
to
80e28e9
Compare
@FRidh I pushed the version that tries to use |
See https://github.com/FRidh/nixpkgs/tree/certbot for what I would say is the preferred solution. |
Agreed. Thanks for the commits! I pushed them to this PR (slightly modifying |
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.
IMO the commits can be squashed into a single one. Do try to get rid of the whitespace changes in pkgs/top-level/python-packages.nix. Other than that it looks good to me.
I'll squash the commits adding the The trailing whitespace cleanups are harder to get rid of, my editor is really persistent on cleaning these up :-D I'll see if I can move them into a individual commit, so |
This can be used to wrap certbot to include some plugins. certbot.withPlugins has a similar calling convention as python*.withPackages: ``` certbot.withPlugins (cp: [ cp.certbot-dns-foo ]) ```
9419e28
to
9d86dea
Compare
Can I use this inside my system configuration? Not sure how to configure it
|
You need to use
Note |
Thanks that worked for me, at least importing the plugin where I have the PR here #190677 Unfortunately it doesn't get recognized yet
|
Let's take this over to the PR, this is unrelated to the introduction of |
Simplifies making use of certbot plugins introduced in #92696.
This can be used to wrap certbot to include some plugins.
certbot.withPlugins has a similar calling convention as python*.withPackages:
Motivation for this change
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)