[Bug 200015] [New port] dns/dnscrypt-wrapper

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=200015

Jason Unovitch <junovitch at freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #157996|0                           |1
        is obsolete|                            |
                 CC|                            |junovitch at freebsd.org
           Assignee|freebsd-ports-bugs at FreeBSD. |junovitch at freebsd.org
                   |org                         |
             Status|New                         |In Progress

--- Comment #3 from Jason Unovitch <junovitch at freebsd.org> ---
Created attachment 160429
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=160429&action=edit
Revised Shar

Hello,
I've attached a revision and I have some questions.

First off, let's not commit an old version to start off. 0.2 was released at
the start of the month. Fortunately this solves the issue regarding the
separate argparse so drop that Makefile code.

Formatting:
- standardize indentation in Makefile
- wrap pkg-descr at 80 lines
- add a '/' to the end of the WWW for a proper URL

Content:
- Don't set PORTREVISION= 1 for a brand new port
- Use %%USERS%% in the rc.d script
- Also use %%ETCDNSCRYPTWRAPPER%% variables rather than
/var/log/dnscrypt-wrapper.log in multiple places in the rc.d script
- Remove the crypt public key lines, per 0.2 release notes:
"Crypt public key is not required, see README.md for detail instructions."

Questions:
1. Is the umask 007 for the keys correct?  What use does the wheel group have
for needing read access to the keys?
2. How did you test it at runtime? It's just hanging when I do the drill shown
on the authors github. Any feedback would be appreciated.

-- 
You are receiving this mail because:
You are the assignee for the bug.