Bug 214515
| Summary: | net-mgmt/observium: fix license; improve post install message; add apache config | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | dirkx | ||||||||
| Component: | Individual Port(s) | Assignee: | freebsd-ports-bugs (Nobody) <ports-bugs> | ||||||||
| Status: | Closed FIXED | ||||||||||
| Severity: | Affects Some People | CC: | dirkx, rakuco | ||||||||
| Priority: | --- | Keywords: | needs-patch, needs-qa | ||||||||
| Version: | Latest | Flags: | koobs:
maintainer-feedback?
(dirkx) koobs: merge-quarterly? |
||||||||
| Hardware: | Any | ||||||||||
| OS: | Any | ||||||||||
| Attachments: |
|
||||||||||
Created attachment 177004 [details]
Updated patch after discussion with Adam Armstrong (the author)
Updated license permissions; removed REVISION 'portlint' complained about.
Final version - approved by the maintainer. End user license is a variant of the QPL. Set it to 'EULA' - as that seemed more appropriate than 'subverting' something like QPL10 or inventing a value. This patch also adds the much needed apache config and improves the post install message a little. Created attachment 177005 [details]
Updated patch - whitespace and dir reference.
Last minute change/cleanup.
You also need to bump PORTREVISION in the patch since the generated package is being changed. Pending maintainer feedback and review as per comment 0 Comment on attachment 177005 [details] Updated patch - whitespace and dir reference. @Maintainer, please 'unobsolete' this attachment when it is considered 'ready' to commit (post review mentioned in comment 0). A patch existing, with maintainer approval are signals that there is nothing (short of additional QA) to commit it. A commit references this bug: Author: swills Date: Tue Aug 22 16:26:14 UTC 2017 New revision: 448544 URL: https://svnweb.freebsd.org/changeset/ports/448544 Log: net-mgmt/observium: multiple changes * Fix license * Improve post install message * Add Apache config PR: 214515 Submitted by: dirkx@webweaving.org (maintainer) (with slight modification) Changes: head/net-mgmt/observium/Makefile head/net-mgmt/observium/files/observium.conf.sample.in head/net-mgmt/observium/files/pkg-message.in head/net-mgmt/observium/pkg-plist Committed, thanks! |
Created attachment 177003 [details] -ruN patch with delta. Apparently we've not been using quite the right license construct for Observium. This patch should fix this. NOT ready for commit into the tree yet - I am reviewing this with the makers of Observium. This patch also adds the much needed apache config and improves the message a wee little.