Skip to content
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

dnscrypt-proxy module points at non-existing manpage #41021

Closed
coretemp opened this issue May 24, 2018 · 3 comments · Fixed by #41039
Closed

dnscrypt-proxy module points at non-existing manpage #41021

coretemp opened this issue May 24, 2018 · 3 comments · Fixed by #41039

Comments

@coretemp
Copy link
Contributor

Issue description

man configuration.nix contains

services.dnscrypt-proxy.extraArgs
           Additional command-line arguments passed verbatim to the daemon. See dnscrypt-proxy(8) for details.

           Type: list of strings

, but there is no man page available.

Steps to reproduce

LC_ALL=C man 8  dnscrypt-proxy
No manual entry for dnscrypt-proxy in section 8

Technical details

  • system: "x86_64-linux"
  • host os: Linux 4.14.41, NixOS, 18.03.git.ee80654 (Impala)
  • multi-user?: yes
  • sandbox: no
  • version: nix-env (Nix) 2.0.1
  • channels(root): "nixos-17.09.2182.7f6f0c49f0"
  • channels(coretemp): "nixos-18.03.132192.ce0d9d638de"
  • nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

CC: @rnhmjoj (author of module)

@dotlambda
Copy link
Member

You have to install the dnscrypt-proxy package for the man page to be available. You can also find it here: https://www.mankier.com/8/dnscrypt-proxy.

@coretemp
Copy link
Contributor Author

@dotlambda You should not close this issue, because it is a bug. If a module documents something, it should deliver on that.

Either the documentation should include the string You need to add dnscrypt-proxy to your environment.systemPackages for the man page to be available. or the module should be fixed to put dnscrypt-proxy into environment.systemPackages.

I see the above two as resolutions to this problem. There might be other resolutions, but closing this issue is not one of them. It's not productive to close valid bugs.

@dotlambda
Copy link
Member

Feel free to open a PR that adds dnscrypt-proxy to environment.systemPackages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants