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.
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!