Bug 228390
| Summary: | dns/unbound: Fix checkconf when unbound_conf is set | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Bernard Spil <brnrd> | ||||
| Component: | Individual Port(s) | Assignee: | Bernard Spil <brnrd> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | jaap | ||||
| Priority: | --- | Keywords: | needs-qa, patch | ||||
| Version: | Latest | Flags: | jaap:
maintainer-feedback+
koobs: merge-quarterly? |
||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
(In reply to Bernard Spil from comment #0) Looks fine to me, go ahead. jaap A commit references this bug: Author: brnrd Date: Mon May 21 19:28:28 UTC 2018 New revision: 470572 URL: https://svnweb.freebsd.org/changeset/ports/470572 Log: dns/unbound: Fix errors when unbound_conf is set PR: 228390 Approved by: maintainer <jaap NLnetLabs nl> MFH: 2018Q2 Changes: head/dns/unbound/files/unbound.in Thanks Jaap! Committed. A commit references this bug: Author: brnrd Date: Tue May 22 16:42:55 UTC 2018 New revision: 470626 URL: https://svnweb.freebsd.org/changeset/ports/470626 Log: dns/unbound: Bump portrevision - Follow-up of r470572 PR: 228390 Reported by: adamw MFH: 2018Q2 Changes: head/dns/unbound/Makefile A commit references this bug: Author: brnrd Date: Wed Jun 6 18:48:09 UTC 2018 New revision: 471876 URL: https://svnweb.freebsd.org/changeset/ports/471876 Log: MFH: r470572 dns/unbound: Fix errors when unbound_conf is set PR: 228390 Approved by: maintainer <jaap NLnetLabs nl> Approved by: ports-secteam (feld) Changes: _U branches/2018Q2/ branches/2018Q2/dns/unbound/files/unbound.in |
Created attachment 193588 [details] svn diff for dns/unbound When unbound_conf is set in rc.conf, there's an ordering error in the rc script. ${unbound_conf} is used in precmd which is defined before unbound_conf is defined effectively always testing against the default unbound.conf which is compiled-in. ``` dns/unbound: Fix errors when unbound_conf is set PR: Approved by: (maintainer) ```