Created attachment 160462 [details]
Better handling of configuration files by not including a really old validator.conf.sample in files and instead patching the one included with the software to begin with. Also include their httpd.conf sample instead of including that in pkg-message.
Also made use of the WWWDIR definition because that probably should've been used from the start anyways, and also fixing where the check script itself looks for the files.
Bumping the PORTREVISION because of how big the change actually is, despite only changing what files are installed.
Poudriere log in the next reply.
Created attachment 160463 [details]
Can you silence the portlint warnings and submit new patch afterwards?
The only portlint warnings I get (when I do portlint -abt) are 2 for "unknown" pkg-plist directives and 1 for LICENSE.
From what I've read in the porter's handbook, those pkg-plist directives I've used are valid and there are no issues when installing the package. I think portlint just doesn't know about those directives and that is why it complains.
As for LICENSE, the framework for LICENSE still has yet to be described in the porter's handbook and I do not feel like tracking down what license is used or how to describe it in the Makefile. Maybe if there was some actual documentation to it, I would do it, but I do not have the time to waste on something that isn't even being given attention to in the porter's handbook when everything else is.
A commit references this bug:
Date: Sun Nov 8 11:33:46 UTC 2015
New revision: 401050
www/validator: various improvements
- Provide patched validator.conf.sample from original sources
- Remove httpd.conf sample file from pkg-message
- Use WWWDIR variable
Submitted by: Naram Qashat <email@example.com> (maintainer)